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

Bug #904368 reported by Andrea Cimitan
34
This bug affects 6 people
Affects Status Importance Assigned to Milestone
Ayatana Design
New
Undecided
Unassigned
Session Menu
Confirmed
Low
Charles Kerr
indicator-session (Ubuntu)
Confirmed
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)

Revision history for this message
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
Revision history for this message
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)
Changed in indicator-session:
assignee: nobody → charles (charlesk)
Revision history for this message
Adolfo Jayme Barrientos (fitojb) 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
Revision history for this message
Adolfo Jayme Barrientos (fitojb) wrote :

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

What do you think it's the better solution?

Revision history for this message
Sam_ (and-sam) wrote :

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

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

Duplicates of this bug

Other bug subscribers

Related blueprints

Remote bug watches

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