[Breeze] [Bug 373088] Current tab highlighting counter-intuitive when using dark themes
https://bugs.kde.org/show_bug.cgi?id=373088 Maximiliano Curia changed: What|Removed |Added CC||m...@debian.org --- Comment #15 from Maximiliano Curia --- (In reply to Hugo Pereira Da Costa from comment #13) > (In reply to Christoph Feck from comment #12) > > According to the bug's history, it was you who removed you as the assignee > > :) > > > > https://bugs.kde.org/show_activity.cgi?id=373088 > > Well, I guess that must be the other me, taking action when I turn my back. > More seriously, about to push. > - to master > - to Plasma/5.9 (since this is a bug right ?) > Should it also be pushed to Plasma/5.8 ? (LTS) ? Yes, please, the original report was reported against 5.8.4, and the https://bugs.debian.org/866996 follow ups indicate that users of 5.8 are still affected by this. Happy hacking, -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 373088] Current tab highlighting counter-intuitive when using dark themes
https://bugs.kde.org/show_bug.cgi?id=373088 Hugo Pereira Da Costa changed: What|Removed |Added Latest Commit||https://commits.kde.org/bre ||eze/13c049c6fba95cf58de9bcc ||3f61df56153b7c54f Resolution|--- |FIXED Status|CONFIRMED |RESOLVED --- Comment #14 from Hugo Pereira Da Costa --- Git commit 13c049c6fba95cf58de9bcc3f61df56153b7c54f by Hugo Pereira Da Costa. Committed on 21/02/2017 at 09:30. Pushed by hpereiradacosta into branch 'Plasma/5.9'. Use QPalette::Shadow instead of QPalette::Text to darken inactive tabs M +1-1kstyle/breezestyle.cpp https://commits.kde.org/breeze/13c049c6fba95cf58de9bcc3f61df56153b7c54f -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 373088] Current tab highlighting counter-intuitive when using dark themes
https://bugs.kde.org/show_bug.cgi?id=373088 --- Comment #13 from Hugo Pereira Da Costa --- (In reply to Christoph Feck from comment #12) > According to the bug's history, it was you who removed you as the assignee :) > > https://bugs.kde.org/show_activity.cgi?id=373088 Well, I guess that must be the other me, taking action when I turn my back. More seriously, about to push. - to master - to Plasma/5.9 (since this is a bug right ?) Should it also be pushed to Plasma/5.8 ? (LTS) ? -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 373088] Current tab highlighting counter-intuitive when using dark themes
https://bugs.kde.org/show_bug.cgi?id=373088 --- Comment #12 from Christoph Feck --- According to the bug's history, it was you who removed you as the assignee :) https://bugs.kde.org/show_activity.cgi?id=373088 -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 373088] Current tab highlighting counter-intuitive when using dark themes
https://bugs.kde.org/show_bug.cgi?id=373088 Hugo Pereira Da Costa changed: What|Removed |Added CC||hugo.pereira.da.costa@gmail ||.com Assignee|plasma-de...@kde.org|hugo.pereira.da.costa@gmail ||.com Component|general |QStyle --- Comment #11 from Hugo Pereira Da Costa --- Will fix. Thanks Christoph for the suggested patch, it is a good one. (would have implemented earlier if i had been in the assignee list). -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 373088] Current tab highlighting counter-intuitive when using dark themes
https://bugs.kde.org/show_bug.cgi?id=373088 --- Comment #10 from Christoph Feck --- (or "QPalette::Shadow" at your convenience...) -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 373088] Current tab highlighting counter-intuitive when using dark themes
https://bugs.kde.org/show_bug.cgi?id=373088 --- Comment #9 from Christoph Feck --- According to the code, the color that is used to darken the inactive tabs is the WindowText color. With reverse (dark) themes, this color is bright, so instead of darkening, the inactive tabs become lighter. Suggested patch: Change "QPalette::WindowText" to "QPalette::Dark" in breezestyle.cpp, line 5549. -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 373088] Current tab highlighting counter-intuitive when using dark themes
https://bugs.kde.org/show_bug.cgi?id=373088 --- Comment #8 from Francis Herne --- Tangentially, I looked through the colours in Breeze-dark, and can't find any one that matches the inactive tab colour. It's not clear where this is colour derived from, or how to modify it. (also, I messed up my previous comment slightly; part of one entry is duplicated. It's 2am here, time to sleep...) -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 373088] Current tab highlighting counter-intuitive when using dark themes
https://bugs.kde.org/show_bug.cgi?id=373088 --- Comment #7 from Janek Bevendorff --- I agree that Breeze is the nicest theme of all those. But the inverted colors are clearly confusing and counter-intuitive. I'm not quite sure where the error lies, though. This weird tab coloring also appears with other dark color schemes, such as various Krita color schemes. So it could either be that they are all just based on each other and inherit the same bug or that it is really an issue with the Breeze widget style. The fact that changing the widget style leads to correct tab coloring lets strongly assume it is. -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 373088] Current tab highlighting counter-intuitive when using dark themes
https://bugs.kde.org/show_bug.cgi?id=373088 --- Comment #6 from Francis Herne --- Created attachment 102846 --> https://bugs.kde.org/attachment.cgi?id=102846&action=edit Comparison of tabs in different widget styles -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 373088] Current tab highlighting counter-intuitive when using dark themes
https://bugs.kde.org/show_bug.cgi?id=373088 --- Comment #5 from Francis Herne --- I think your statement is incorrect - no other widget style installed on my system displays tabs with reversed colours, using Breeze-dark or other dark schemes. In the attachment, all using Breeze-dark, top to bottom: - QtCurve (default settings): active tab is lighter, taller and has red close icon. - Oxygen: active tab is lighter, taller all using Breeze-dark, top to bottom: - QtCurve (default settings): active tab is lighter, taller and has red close icon. - Win9x (bleh): active tab has same colour, taller, has red close icon. - Fusion: active tab is lighter and taller. All close buttons identical (red). - Breeze: active tab is much darker, same height. All close buttons identical (grey). To be honest, I find Breeze's tabs clearer than the other variants with this colour scheme - ignoring distinct close-icon colours which make a huge difference - but the inverse-colour behaviour is clearly influenced by Breeze QtStyle and is reversed from all other styles. -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 373088] Current tab highlighting counter-intuitive when using dark themes
https://bugs.kde.org/show_bug.cgi?id=373088 Hugo Pereira Da Costa changed: What|Removed |Added CC|hugo.pereira.da.costa@gmail | |.com| -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 373088] Current tab highlighting counter-intuitive when using dark themes
https://bugs.kde.org/show_bug.cgi?id=373088 Hugo Pereira Da Costa changed: What|Removed |Added Assignee|hugo.pereira.da.costa@gmail |plasma-de...@kde.org |.com| Component|QStyle |general --- Comment #4 from Hugo Pereira Da Costa --- I mean: either modify the official breeze-dark theme, or you modify the theme you use to something that suits you better. -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 373088] Current tab highlighting counter-intuitive when using dark themes
https://bugs.kde.org/show_bug.cgi?id=373088 Hugo Pereira Da Costa changed: What|Removed |Added CC||hugo.pereira.da.costa@gmail ||.com --- Comment #3 from Hugo Pereira Da Costa --- Its related to color scheme, not widget style. Dark themes are usually "reverse video" themes, where brighter becomes darker and darker lighter (see for instance how the "text view" is darker). That's what happening here. What's needed is to modify the color scheme. not the widget style. -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 373088] Current tab highlighting counter-intuitive when using dark themes
https://bugs.kde.org/show_bug.cgi?id=373088 --- Comment #2 from Janek Bevendorff --- Well, no matter which module this bug is in, it should be fixed and I don't want to change my widget style just because I use a different color theme. -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 373088] Current tab highlighting counter-intuitive when using dark themes
https://bugs.kde.org/show_bug.cgi?id=373088 Francis Herne changed: What|Removed |Added CC||m...@flherne.uk Version|git master |5.8.4 Status|UNCONFIRMED |CONFIRMED Component|UI: general |QStyle Product|kdevelop|Breeze Assignee|kdevelop-bugs-n...@kde.org |hugo.pereira.da.costa@gmail ||.com Ever confirmed|0 |1 --- Comment #1 from Francis Herne --- This is unrelated to KDevelop - the Breeze widget style colours tabs this way in all applications. Try using Fusion widgets with a dark theme, you'll see the active tab is slightly darker and taller. And yes, it confuses me too. -- You are receiving this mail because: You are watching all bug changes.