Opened 7 years ago

Last modified 5 years ago

#4301 new defect

Frame: revisit buttons

Reported by: manuq Owned by: erikos
Priority: Normal Milestone: Unspecified
Component: Sugar Version: Unspecified
Severity: Unspecified Keywords: theme, patch
Cc: garycmartin Distribution/OS: Unspecified
Bug Status: Unconfirmed

Description

There is a mix of a) tool buttons and b) tray icons.

Tool buttons have mouse-over feedback, as a black background. Tray icons don't.

In the bottom panel, we are mixing tool button for wireless AP and olpc-mesh AP, with trayicons for the others (battery, speech, etc).

Change History (6)

comment:1 Changed 6 years ago by dnarvaez

  • Milestone changed from 1.0 to Unspecified

comment:2 Changed 5 years ago by godiard

Referenced in #3293 too

comment:3 Changed 5 years ago by walter

  • Priority changed from Unspecified by Maintainer to Normal

comment:4 Changed 5 years ago by godiard

  • Priority changed from Normal to Urgent

In Sugar 0.102, there are a regression with these buttons. No border is displayed on hover or when the palette is opened. This add to the prelight issue detailed on #3293.

comment:5 Changed 5 years ago by godiard

  • Priority changed from Urgent to Normal

Regression was produced by commit b9d6b628a98fc17f5e92a344450b6336f1a2b2ce

Sent a pr to solve that issue (button border not displayed, palette showing a border around). Change this ticket priority, because the remaining issue is not very noticeable.
A initial implementation of a fix is here [1] but should use Icon instead of PulseIcon.

[1] http://www.google-melange.com/gci/task/view/google/gci2013/5868450979250176

comment:6 Changed 5 years ago by godiard

  • Keywords patch added
Note: See TracTickets for help on using tickets.