Session Menu

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

Bug #904368 reported by Andrea Cimitan on 2011-12-14

This bug affects 6 people

	Status	Importance	Assigned to
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)

Tags:

Revision history for this message

Bilal Akhtar (bilalakhtar) wrote on 2011-12-20:

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 on 2012-01-04:

orange-on-red.ogv Edit (4.1 MiB, video/ogg)

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)

Revision history for this message

Adolfo Jayme Barrientos (fitojb) wrote on 2012-01-14:

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 on 2012-01-14:

...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 on 2012-01-14:

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

Andrea Cimitan (cimi) on 2012-05-02

tags:

added: euv