TextField's placeHolderText invisible inside a dialog
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| | Ubuntu UX |
High
|
Olivia Caro de Diego | ||
| | ubuntu-ui-toolkit (Ubuntu) |
High
|
Christian Dywan | ||
| | Vivid |
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
- PS Jenkins bot: Approve (continuous-integration) on 2015-06-11
- Zsombor Egri (community): Approve on 2015-06-10
- Olivia Caro de Diego: Pending requested 2015-06-01
-
Diff: 40 lines (+6/-2)3 files modifiedexamples/ubuntu-ui-toolkit-gallery/Dialog.qml (+4/-0)
modules/Ubuntu/Components/Themes/SuruDark/1.2/Palette.qml (+1/-1)
modules/Ubuntu/Components/Themes/SuruDark/1.3/Palette.qml (+1/-1)
| 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 |
| Christian Dywan (kalikiana) wrote : | #2 |
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 |
| Michael Zanetti (mzanetti) wrote : | #3 |
Screenshot: http://
Note, the two visible labels have manually set the color to "black".
Here's the code for the dialog: http://
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
| Christian Dywan (kalikiana) wrote : | #5 |
I can't reproduce it with the above snippet, both on rtm and stagiging the text is readable.
| Christian Dywan (kalikiana) wrote : | #6 |
| Christian Dywan (kalikiana) wrote : | #7 |
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.
| Christian Dywan (kalikiana) wrote : | #8 |
So what happens is that the backgroundColor causes the automatic switch to SuruDark which has a bad normal.
| Changed in ubuntu-ui-toolkit (Ubuntu): | |
| status: | Incomplete → Confirmed |
| importance: | Undecided → High |
| status: | Confirmed → In Progress |
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 |
| Launchpad Janitor (janitor) wrote : | #10 |
This bug was fixed in the package ubuntu-ui-toolkit - 1.3.1534+
---------------
ubuntu-ui-toolkit (1.3.1534+
[ Christian Dywan ]
* Use TextField.write rather than keyboard.type.
* Update normal.
* 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-
* 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.
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
Orientation
[ 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 |


Hi Christian,
Would you mind to provide screenshots, please?
Thank you,
Olivia