Popovers under SuruGradient theme are white-on-white

[Michał Sawicz]

The recent searches in dash are all white - I expect this to be a theming / palette impact.

[Michał Sawicz (saviq)]

The example from Popover documentation [1] is white-on-white under SuruGradient theme:
http://pastebin.ubuntu.com/5914173/

See the attached image.

[1] http://developer.ubuntu.com/api/ubuntu-12.10/qml/mobile/qml-ubuntu-components-popups0-popover.html

[Michał Sawicz (saviq)]

Hey, can we please get that looked at? We're adding launcher quicklists now so it's going to be much more visible...

[Jouni Helminen (jounihelminen)]

really need this fixed please, the previous dash searches are invisible/unusable right now, just shows a large white popover with white text

[Bill Filler (bfiller)]

this is critical for new dialer, messaging and address book apps. they all display popovers and use the new theme. you can't read the text in the popover currently

[Tim Peeters (tpeeters)]

Unfortunately, this is not a 5min fix. I had a look at it and the problem is that the ListItems are not aware of their background color (in this case defined by the Popover). Somehow we need to make it aware that is inside something with an "overlay" color (in the LabelVisual) so that it uses the foregroundText color belonging to the popover.

[PS Jenkins bot (ps-jenkins)]

Fix committed into lp:ubuntu-ui-toolkit at revision None, scheduled for release in ubuntu-ui-toolkit, milestone Unknown

[Tim Peeters (tpeeters)]

A quick fix was committed, but the real issue is that the same text color for list items does not work on the background (dark) and popover (white background) at the same time. I think a good solution would be that for dark themes, the popovers should also have a dark background. This is something for design to look at.

[Jouni Helminen (jounihelminen)]

Design comment - I don't think we want the popovers to change between light/dark depending on what app is being used - they should be white by default, and the text inside them should be dark by default so that you can read it.

Can you not override the white listitem text colour with dark grey by default when using (white) popovers?

[Launchpad Janitor (janitor)]

Status changed to 'Confirmed' because the bug affects multiple users.

[Launchpad Janitor (janitor)]

This bug was fixed in the package ubuntu-ui-toolkit - 0.1.46+13.10.20130830.1-0ubuntu1

---------------
ubuntu-ui-toolkit (0.1.46+13.10.20130830.1-0ubuntu1) saucy; urgency=low

  [ tpeeters ]
  * Quick fix, completely untested. Might mess up apps with list items
    that do not use popovers. Have a look to see if you can use this
    code and it doesn't break anything. (LP: #1205094)
  * Clean toolbar code by removing deprecated properties and updating
    tests.

  [ <email address hidden> ]
  * ajusted the buttons laytout to make it fit to phone screen size.
    https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1218749 . (LP:
    #1218749)

  [ Ubuntu daily release ]
  * Automatic snapshot from revision 731
 -- Ubuntu daily release <email address hidden> Fri, 30 Aug 2013 12:44:49 +0000