Header divider is invisible on very dark backgrounds
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| | Ubuntu UX |
High
|
Jouni Helminen | ||
| | ubuntu-ui-toolkit (Ubuntu) |
High
|
Tim Peeters | ||
Bug Description
The color for the header divider is calculated by Qt.darker(
Ideally the app developer could also override the color for the divider so that it can match the foregroundColor, meaning the divider would be the same color as the text and icons inside the header.
| Changed in ubuntu-ui-toolkit (Ubuntu): | |
| status: | New → Confirmed |
| Tim Peeters (tpeeters) wrote : | #1 |
| Changed in ubuntu-ui-toolkit (Ubuntu): | |
| assignee: | nobody → Tim Peeters (tpeeters) |
| tags: | added: design-ync |
| tags: |
added: design-sync removed: design-ync |
| Changed in ubuntu-ux: | |
| assignee: | nobody → Jouni Helminen (jounihelminen) |
| status: | New → Triaged |
| importance: | Undecided → High |
| Changed in ubuntu-ui-toolkit (Ubuntu): | |
| importance: | Undecided → High |
| Jouni Helminen (jounihelminen) wrote : | #2 |
App devs should definitely be able to define the divider colour so they can match it with their app.
If we can do that automatically, even better.
We are in the process of redesigning the large divider( with a shadow and highlight) and reducing it to single thin line - should have that work approved in 2 weeks. Probably makes sense to do the implementation after that
| Changed in ubuntu-ux: | |
| status: | Triaged → Fix Committed |
| Tim Peeters (tpeeters) wrote : | #3 |
This is fixed with the new header and new theming with palette.
| Changed in ubuntu-ui-toolkit (Ubuntu): | |
| status: | Confirmed → Fix Released |


Currently we have this in MainView.qml:
I propose, as Michael said, to use a "darkness threshold" and if the color is too dark, use dividerColor: Qt.lighter(...) instead of Qt.darker().
Ideally we should also make all the color properties available so that they can be change din the header style.