Font Colour for Time List in "Week" and "Day" view lack contrast.

Bug #1357260 reported by Kill Animals
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ubuntu Calendar App
Fix Released
Low
Nekhelesh Ramananthan

Bug Description

http://i.imgur.com/CWHKMf2.png

An aesthetic complaint:

From my point of view, the font struck me as not having enough contrast, causing me to want to squint. Changing the colour to "UbuntuColors.coolgrey" I would imagine would work better.

============ Solution (nik90) ===============
Change the time text color to "UbuntuColors.coolGrey" to provide a better contrast with the background.

Related branches

Revision history for this message
Mihir Soni (mihirsoni) wrote :

Hi, we would like to have designer inputs on this.

Changed in ubuntu-calendar-app:
importance: Undecided → Low
status: New → Confirmed
Revision history for this message
Kill Animals (kill-animals) wrote :

I'll be working on another bug that may solve this issue:

https://code.launchpad.net/~akiva/ubuntu-calendar-app/current-hour-highlighted/+merge/232950

I'll be creating two branches to gauge the aesthetic preferences. Stay tuned.

Changed in ubuntu-calendar-app:
assignee: nobody → Akiva (akiva)
Revision history for this message
Kill Animals (kill-animals) wrote :

http://imgur.com/YWshkLw

Please leave a comment on whether you like any of these designs

Revision history for this message
Mihir Soni (mihirsoni) wrote :

I would prefer 2nd option.

Revision history for this message
Kunal Parmar (pkunal-parmar) wrote :

to me 3'rd seems good,

 as we need some kind of separator between individual hours and also identifies for working hours, 3rd seems to offer both.

1st and 4th also offer same,but to me color is not matching current gray theme.

BTW, we need to make working hours configurable,.

Revision history for this message
Kill Animals (kill-animals) wrote :

I redid the screenshots, with some events in the page. Take a look:

http://imgur.com/a/FIFRy#0

One option may to be, that instead of altering colours, we could put lines between the rectangle.

One thing I would like to see actually, is the time label sitting on top of the line, instead of in the centre, making easier to tell what time the line represents.

Changed in ubuntu-calendar-app:
milestone: none → rtm14
status: Confirmed → In Progress
description: updated
Revision history for this message
Nekhelesh Ramananthan (nik90) wrote :

Guys, let's not do any drastic background or font-color changes please. This bug report was originally about a lack of contrast between the text and the background. So let's just solve that by increasing the darkness of the font color to UbuntuColors.coolGrey. The reason I say this is because the design proposals by Akiva require a design team to look into it and approve. However I don't see that happening considering that the design team is busy atm.

The current grey background was given to us by Alan Pope and a designer. Let's just stick to it. I hope I don't come across as being rude as that's not my intention.

@Akiva, if you want create another bug report about an improved design for the timeline view with your design proposal and we can take that up with the designer post-rtm. But thanks a lot for your contributions and efforts.

Revision history for this message
Kill Animals (kill-animals) wrote :

I should point out that Popey did comment well on the grey backgrounds, as it did help the events stick out nicely.

http://i.imgur.com/ReqZl60.png

As said, I will put something together with that patch you linked me in IRC, and we can just use our discretion from there.

Changed in ubuntu-calendar-app:
assignee: Akiva (akiva) → Nekhelesh Ramananthan (nik90)
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :

Fix committed into lp:ubuntu-calendar-app at revision 460, scheduled for release in ubuntu-calendar-app, milestone rtm14

Changed in ubuntu-calendar-app:
status: In Progress → Fix Committed
Changed in ubuntu-calendar-app:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.