GNOME Bugzilla – Bug 642194
Update calendar font styles to match mockups
Last modified: 2011-02-23 01:35:02 UTC
Noticed that there was some clarity problems with things like being able to see at a glance which days had events. Looking at the mockup here: http://git.gnome.org/browse/gnome-shell-design/plain/mockups/static/date-n-time.png I noticed that the coloration and some of the sizing/boldness is different than it currently is, fixing that issue. Quick patch to follow shortly.
Created attachment 180740 [details] [review] Update calendar font style Now that we are using Cantrell as the default UI font, some clarity was reduced in using the calendar. Mockups already address this so change the font styles in CSS to match the mockups.
Created attachment 180747 [details] [review] Update calendar font style Missed a bold. Also misspelled Cantarell in the commit message.
Looks good to me. The very last thing will be the active day indicator (bottom border + radial gradient svg) but that has to happen everywhere else. I saw florian working on addressing that in the meantime.
(In reply to comment #3) > Looks good to me. The very last thing will be the active day indicator (bottom > border + radial gradient svg) but that has to happen everywhere else. I saw > florian working on addressing that in the meantime. Yes, I have that as part of the panel style update.
Review of attachment 180747 [details] [review]: Noticed a bug when I'd tried applying this - if you have stuff under "This Week", because you moved "This week" to the right using padding on the header, you get: This week Thu 11:00 Team Meeting
(In reply to comment #5) > Review of attachment 180747 [details] [review]: > > Noticed a bug when I'd tried applying this - if you have stuff under "This > Week", because you moved "This week" to the right using padding on the header, > you get: > > This week > Thu 11:00 Team Meeting I see now. I didn't have a non-all day appointment when I was testing.
Created attachment 180956 [details] [review] Update calendar font style This should fix the situation Owen mentioned. It -should- also align the day name with the This Week (something I just noticed in the mockup). Let me know if I broke things horribly (although it looked fine clicking around to a few different days/weeks).
Alignment and padding still could use some improvement to match the mockups, but this looks good. Attachment 180956 [details] pushed as d1eb2b2 - Update calendar font style