iOS Catalyst: UI Updates

samuelclay / NewsBlur

NewsBlur is a personal news reader that brings people together to talk about the world. A new sound of an old instrument.

http://www.newsblur.com

MIT License

6.85k stars 997 forks source link

iOS Catalyst: UI Updates #1442

Closed samuelclay closed 3 years ago

samuelclay commented 3 years ago

This latest beta is incredible! It's ready to roll on TestFlight. I'm wondering how hard it would be to make these changes beforehand. I'd like to update the font, sizes, icons, and a bit of styling all over the app to match the upcoming redesign at beta.newsblur.com.

App font is now Whitney everywhere. I've found that the equivalent size is one px higher. So 14px Helvetica is 15px Whitney.
A few folder icons are updated
A tiny bit of extra vertical space for feeds for easier tapability
Story titles have a bit more breathing room as well
No more uppercase everything. Folder names, menu items, share buttons should all be "Sentence case".
Use icons instead of numbers in the menu "Title | 1 | 2 | 3" /media/img/reader/contentpreview{small,medium,large}.png

Screen Shot 2021-03-26 at Mar 26 2 02 23 PM

Screen Shot 2021-03-26 at Mar 26 2 03 20 PM

Screen Shot 2021-03-26 at Mar 26 2 01 25 PM

Screen Shot 2021-03-26 at Mar 26 3 10 45 PM

Dejal commented 3 years ago

Done. If I've missed any icons, please identify them, preferably by path, or at least by name.

samuelclay commented 3 years ago

Amazingly quick turnaround! Few notes:

Is it just me or are the story titles font sizes a step (s->m, m->l) bigger than the story detail font size? I was at Medium on both and it felt like the story text went smaller.
The descenders on folder titles are cut off. g and p etc
In story titles, the story content preview should be one font size smaller than the title
"Book" vs "Medium": I don't know why but the website's Medium is a bit bolder than the apps. I only use two weights and they should be the same between app and web, but I should look into why
In story titles, on the small image preview, there should be a bit more space to the left of the image
Times should use the new format. "10:00am" instead of "10:00 AM"
Times should now match the color of author in story titles
Share/Train/Save buttons need Sentence case
The feed gradient title should be a nudge taller (2-4px)

Dejal commented 3 years ago

I'll see what I can do later today.

nriley commented 3 years ago

@samuelclay — an issue with using "am" rather than "AM" is that it's not a standard localized time format on iOS (at least for US English; it actually is standard for UK English if you're not using 24-hour time). Do you want NewsBlur to stop using standard time formats for this?

Dejal commented 3 years ago

Yes, I'm inclined to decline the time format change. It uses the system date format, and should continue to do so (e.g. I use 24 hour time).

samuelclay commented 3 years ago

Sure, I'm fine with that

Dejal commented 3 years ago

Try that on for size.