The red color used for restart should not be used on prelight

Reported by Andrea Cimitan on 2011-12-14
34
This bug affects 6 people
Affects Status Importance Assigned to Milestone
Ayatana Design
Undecided
Unassigned
Session Menu
Low
Charles Kerr
indicator-session (Ubuntu)
Low
Unassigned

Bug Description

When the menuitem is selected/prelighet/active (don't remember exactly the state) the text color of the red menuitem should be the normal one, otherwise it could lead to red-on-orange (in the case of ubuntu ambiance theme)

Bilal Akhtar (bilalakhtar) wrote :

Correct, reading red on orange is tough.

Changed in indicator-session:
status: New → Triaged
importance: Undecided → Low
status: Triaged → Confirmed
Changed in indicator-session (Ubuntu):
status: New → Confirmed
importance: Undecided → Low
Sam_ (and-sam) wrote :

Reported the bug against light-themes a while ago.
Bug #871287

No change yet in Precise.

indicator-session:
  Installed: 0.3.7.1-1
  Candidate: 0.3.7.1-1
  Version table:
 *** 0.3.7.1-1 0
        500 http://archive.ubuntu.com/ubuntu/ precise/main amd64 Packages
        100 /var/lib/dpkg/status

light-themes:
  Installed: 0.1.8.25
  Candidate: 0.1.8.25
  Version table:
 *** 0.1.8.25 0
        500 http://archive.ubuntu.com/ubuntu/ precise/main amd64 Packages
        100 /var/lib/dpkg/status

Andrea Cimitan (cimi) on 2012-01-12
Changed in indicator-session:
assignee: nobody → charles (charlesk)
Adolfo Jayme (fitoschido) wrote :

I think using red in a menu item is bad and inconsistent. It seems better to highlight the 'Restart to complete updates' item with *bold type*, and not by changing the color of the text in a way that causes issues like this in themes.

tags: added: needs-design
Adolfo Jayme (fitoschido) wrote :

...needless to say that red text looks broken, also ;-)

What do you think it's the better solution?

Sam_ (and-sam) wrote :

Highlight colour purple, white text on both light-themes?

Andrea Cimitan (cimi) on 2012-05-02
tags: added: euv
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Related blueprints