Evolution Calendar sets wrong foreground colour for calendar items

Bug #329125 reported by Chow Loong Jin
2
Affects Status Importance Assigned to Milestone
Evolution
Fix Released
Medium
evolution (Ubuntu)
Fix Released
Low
Chow Loong Jin

Bug Description

Binary package hint: evolution

When using certain themes, such as dark themes, where the selected text
foreground colour is lighter than the normal text foreground colour, Evolution
Calendar sets a dark foreground colour for calendar items which have a dark
background, and a light foreground colour for calendar items which have a light
background. An example of a theme where this happens is Slickness-Black, foudn
here: http://www.gnome-look.org/content/show.php/Slickness+Black?content=73210

Also, when certain themes have the same foreground colour for both selected and
normal text, Evolution sets the same colour for both dark-background calendar
items and light-background calendar items. An example of one such theme is the
Human theme used by Ubuntu.

Based on this observation, it would seem that Evolution's foreground colour
logic is based on the assumption that the normal text foreground colour is
dark, whereas the selected text foreground colour is light.

TEST CASE:
1. Use a theme that fits either of the above criteria.
2. Pick an existing calendar, and change the background colour of the said calendar, component "Value" to 100. Anything above 70 will do.
3. Go to the next month/week/day, and then back (to make the foreground colour refresh).
4. Observe that, in the case of a dark theme, the foreground will now be a light colour, and that the text is now pretty unreadable.
5. Repeat steps 2 to 3 with value 10. Observe that the foreground will now be a dark colour eventhough the background is dark, so the text is now pretty unreadable as well.
6. Repeat steps 1-5 with the Human theme, and notice that the foreground colour stays the same throughout.

Related branches

Changed in evolution:
assignee: nobody → hyperair
status: New → In Progress
Revision history for this message
Chow Loong Jin (hyperair) wrote :
Changed in evolution:
status: In Progress → Confirmed
Revision history for this message
Pedro Villavicencio (pedro) wrote :

thanks for the bug and your work on it, the patch was committed upstream, marking this as fix committed, thanks.

Changed in evolution:
importance: Undecided → Low
status: Confirmed → Triaged
status: Triaged → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package evolution - 2.25.91-0ubuntu1

---------------
evolution (2.25.91-0ubuntu1) jaunty; urgency=low

  * New upstream release. (LP: #330340)
   - Bug Fixes:
      #333716 : Do not store incomplete or broken files in a cache. (Milan Crha)
      #404232 : Rebuild view immediately for an advanced search too. (Milan Crha)
      #548623 : No auto-proxy at the moment. (Milan Crha)
      #554458 : Make content type from a mime type first. (Milan Crha)
      #555888 : Do not overwrite user settings with system proxy settings. Register
      signals at the end, when all widgets are read. Hide socks settings, it is not
      used at the moment. Use proxy configurable within Evolution. (Milan Crha)
      #564229 : Do not use uninitialized variable. (Milan Crha)
      #567089 : Do not crash when no From set yet. (Milan Crha)
      #568176 : Make the migration dialog look better. (Gilles Dartiguelongue)
      #569700 : Stop reading from a network when operation was canceled. (Milan Crha)
      #569986 : Crash while printing (Srinivasa Ragavan) (LP: #327383)
      #570364 : Mark a string for translation (Gabor Kelemen)
      #571625 : Use black/white for foreground instead of active/inactive text color
      (Chow Loong Jin) (LP: #329125)
   - Other contributions:
      Use -mwindows on Windows to build a "GUI" executable. (Tor Lillqvist)
      Do not use non-zero page size in an adjustment, it's deprecated now. (Milan Crha)
      Format signature/encryption information nicer. (Milan Crha) (LP: #60608)
      Function-declaration compiler warning. (Suman Manjunath)
      Remove unneeded #include <libart_lgpl/...>. (Matthew Barnes)
      Substitute some deprecated gtk functions. (Andre Klapper)
  * debian/control:
   - Split documentation into seperate locale packages. (LP: #272779)
   - Replace Recommends on yelp in evolution-common with
     evolution-documentation which is Provided by the
     evolution-documentation-{*} packages.
   - Make evolution-documentation-{*} Depend on yelp and
     Recommend evolution-documentation-en as a fall back.
  * Create evloution-documentation-{cs, de, el, en,
    es, fr, mk, oc, ru, and sv}.install files.

 -- Andrew Starr-Bochicchio <email address hidden> Fri, 13 Feb 2009 13:38:07 -0500

Changed in evolution:
status: Fix Committed → Fix Released
Changed in evolution:
status: Unknown → Fix Released
Changed in evolution:
importance: Unknown → Medium
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.