TextField's placeHolderText invisible inside a dialog

Bug #1413801 reported by Michael Zanetti
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Ubuntu UX
Fix Committed
High
Olivia Caro de Diego
ubuntu-ui-toolkit (Ubuntu)
Fix Released
High
Cris Dywan
Vivid
New
Undecided
Unassigned

Bug Description

Using a dark background in an application, a Dialog and a TextField inside that Dialog renders the placeholderText nearly invisible.

Related branches

Zsombor Egri (zsombi)
Changed in ubuntu-ui-toolkit (Ubuntu):
status: New → Triaged
assignee: nobody → Christian Dywan (kalikiana)
Changed in ubuntu-ux:
status: New → Triaged
assignee: nobody → Olivia Caro de Diego (oliviacaroddiego)
importance: Undecided → High
Revision history for this message
Olivia Caro de Diego (oliviacaroddiego) wrote :

Hi Christian,

Would you mind to provide screenshots, please?

Thank you,
Olivia

Revision history for this message
Cris Dywan (kalikiana) wrote :

I'll pass this request to Michael. I'm assigned to this report because it affects TextField but I haven't actually looked into it.

Changed in ubuntu-ui-toolkit (Ubuntu):
status: Triaged → Incomplete
Revision history for this message
Michael Zanetti (mzanetti) wrote :

Screenshot: http://i.imgur.com/Pma8cs6.png

Note, the two visible labels have manually set the color to "black".

Here's the code for the dialog: http://paste.ubuntu.com/11077309/

Revision history for this message
Olivia Caro de Diego (oliviacaroddiego) wrote :

Hi guys,

The issue in this case is the text color.

Can we use this color instead, please? ----> #5D5D5D

Would you mind to provide a screenshot when ready, please?

Thank you very much, indeed.
Olivia

Revision history for this message
Cris Dywan (kalikiana) wrote :

I can't reproduce it with the above snippet, both on rtm and stagiging the text is readable.

Revision history for this message
Cris Dywan (kalikiana) wrote :
Revision history for this message
Cris Dywan (kalikiana) wrote :

I pondered on this and found how to reproduce afterall: set any backgroundColor on the MainView. Whether it's black, green or orange the placeholder is no longer visible.

Revision history for this message
Cris Dywan (kalikiana) wrote :

So what happens is that the backgroundColor causes the automatic switch to SuruDark which has a bad normal.backgroundColor.

Changed in ubuntu-ui-toolkit (Ubuntu):
status: Incomplete → Confirmed
importance: Undecided → High
status: Confirmed → In Progress
Revision history for this message
Olivia Caro de Diego (oliviacaroddiego) wrote :

Hi Christian,

Would you mind to change the opacity to 50% on #5d5d5d, please?

I was checking with Jouni and this should be ok like this.

Thanks for all,
Olivia

Changed in ubuntu-ux:
status: Triaged → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-ui-toolkit - 1.3.1534+15.10.20150625-0ubuntu1

---------------
ubuntu-ui-toolkit (1.3.1534+15.10.20150625-0ubuntu1) wily; urgency=medium

  [ Christian Dywan ]
  * Use TextField.write rather than keyboard.type.
  * Update normal.backgroundText in SuruDark to #5D5D5D. Fixes LP: #1413801.
  * Fix "runtest.sh: line 78: [: -eq: unary operator expected]".
  * Consider caret visibility to decide text menu anchor. Fixes LP: #1462419.
  * Remove text from header textinput AP test QML. Fixes LP: #1459632.
  * No deprecation notes for our own components. Fixes LP: #1457528.

  [ Leonardo Arias Fonseca ]
  * Add url protocols to the fake application.
  * Adds a dependency to url-dispatcher-tools in ubuntu-ui-toolkit-autopilot.
  * Use the test launcher for all the toolkit autopilot tests.

  [ Zsombor Egri ]
  * Remove property bindings before property write does it. Fixes LP: #1462738.
  * Introducing StyleHints to provide style specific property changes.
  * Remove ThemeSettings.createStyleComponent() and add styleName to StyledItem.
    Fixes LP: #1458845.

  [ Loïc Molinari ]
  * [UbuntuShape] Computed dfdt factors based on window content orientation.
  * Previous logic was based on screen orientation which is not in sync with
    OrientationHelper.qml. Fixes LP: #1455567.

  [ Timo Jyrinki ]
  * Change -html documentation runtime dependencies into Suggests only to limit
    ubuntu-sdk download size. We're switching to offering only .qch offline
    documentation in SDK (inside of it), and they're being added to the seed.
    Fixes LP: #1461034.
  * Licensecheck started complaining about a person name, change to Canonical.
    Fixes LP: #1464549

  [ Andrea Bernabei ]
  * Modify ListItem.Caption label margins. Fixes LP: #1314672.

  [ Tim Peeters ]
  * Introduce IconButtonStyle.

 -- CI Train Bot <email address hidden> Thu, 25 Jun 2015 04:37:52 +0000

Changed in ubuntu-ui-toolkit (Ubuntu):
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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