View Issue Details

IDProjectCategoryView StatusLast Update
0015968ScribusUser Interfacepublic2024-01-14 18:50
Reportermedmedin2014 Assigned To 
PriorityhighSeverityminorReproducibilityalways
Status newResolutionopen 
PlatformUbuntuOS Version19.10 
Product Version1.5.5 
Summary0015968: Disabled and enabled icon in dark mode look the same
DescriptionWhen using Breeze theme with dark icon, the disabled icon on toolbar look the same as the enabled ones which makes the work disturbing and I should hover my mouse on each icon to see if it's enabled or not.
TagsNo tags attached.
PatchNo

Activities

medmedin2014

2019-11-28 12:54

reporter  

Screenshot_20191128_134814.png (58,556 bytes)   
Screenshot_20191128_134814.png (58,556 bytes)   

Luna Nightshade

2020-11-02 18:28

reporter   ~0048297

Is it possible to load a different graphic, if the icon is disabled? (either use a different folder, or prepend a short "DISABLED" to the icon's filename?)

cbradney

2023-12-20 21:38

administrator   ~0050660

Likely fixed, but assigning to Martin to check

medmedin2014

2024-01-02 21:27

reporter   ~0050766

The problem is still visible in 1.6.0 with 1.5.1 Dark icons.

Operating System: Manjaro Linux
KDE Plasma Version: 5.27.10
KDE Frameworks Version: 5.113.0
Qt Version: 5.15.11
Kernel Version: 6.6.9-1-MANJARO (64-bit)
Graphics Platform: Wayland
text1-2-9.png (177,403 bytes)   
text1-2-9.png (177,403 bytes)   

cbradney

2024-01-02 21:44

administrator   ~0050768

But in 1.7.0.svn
image.png (25,054 bytes)   
image.png (25,054 bytes)   

Luna Nightshade

2024-01-02 21:58

reporter   ~0050769

Okay. So, after looking into things a bit I think the following is happening: Disabled Icons are rendered as greyscale, the problem is that the dark icons are already grey.

I see the following solutions here: Either introduce more color to the Dark Icon theme, something I could probably work on, or change how the Icons are managed within Scribus when they're disabled; which would mean getting into the code. And then designing/editing icons and save them for the disabled state.

The former would probably be quicker. Give me a few minutes to create a 1.6.1 dark theme – i have ideas that will probably not look too shabby.

Luna Nightshade

2024-01-02 22:14

reporter   ~0050771

This is now probably moot, since 1.7.0.svn has a nice new icon theme, but for the old ones, see the attachment.
1_6_0_dark.zip (965,291 bytes)

medmedin2014

2024-01-05 15:40

reporter   ~0050822

I extracted and copied "1_6_0_dark" folder to "/usr/share/scribus/icons/"
and added "/usr/share/scribus/icons/1_6_0_dark.xml"

then "Scribus 1.6.0 Dark" entry appeared in Preferences, but even if I select it the icons are still looking the same.

Luna Nightshade

2024-01-05 16:49

reporter   ~0050823

Forgot to edit the XML-file. Right. Changed it in the attachment.
The old XML is still pointing to the 1_5_1_dark folder, which is probably why the icons don't change.
1_6_0_dark-2.zip (965,290 bytes)

Luna Nightshade

2024-01-05 16:51

reporter   ~0050824

By the way, there should be yellow highlights when the icons are active. When they're inactive they'll still look grey, because scribus is rendering them in software to greyscale.

cbradney

2024-01-05 17:21

administrator   ~0050826

@Luna: What did you change? (I have not tested) If we brought that version in, would we also need new light icons? It would not make much sense to include 2 dark palettes of icons. (Plus, need to confirm if your changes are licence compatible with existing)

medmedin2014

2024-01-05 17:26

reporter   ~0050827

The solution to color (a part of or all of) a limited number of icons as yellow is not working for all toolbar icons.

Some enabled icons are still looking gray like disabled ones.

Why not color all enabled icons by pure white and disabled ones by gray.
image1-6.png (153,606 bytes)   
image1-6.png (153,606 bytes)   

medmedin2014

2024-01-05 17:32

reporter   ~0050828

@cbradney Can we safely use 1.7.0 icons in Scribus 1.6.0 ?

cbradney

2024-01-05 17:45

administrator   ~0050830

No, 1.7.0 icons are SVG based so not directly. If someone, maybe @nitramr or some with him, want to work on a single use export of those icons for the right sizes, plus all the renaming, we could use them, yes.

nitramr

2024-01-05 18:59

developer   ~0050832

QIcon supports different images for different icon states, i.e. "normal" and "disabled". If no image is specified for a disabled state, Qt is generating this image based on the UI theme palette colors. Perhaps the color contrast between "normal" and "disabled" is not much in Breeze theme (and in my case Adwaita dark too).

I see 3 options how we can fix that.

1. Additional "disabled icon set"
We can create a darker greyscale version of all existing icons and add them to the icon theme. The IconManager can be extended to add an image for "normal" and another one for "disabled" state.

2. Automatic generated disabled state
similar to 1) the IconManager could render the disabled state itself, so we don't need additional images in icon set. In QIcon code I found this:

QStyleOption opt(0);
opt.palette = QApplication::palette();
QPixmap generated = QApplication::style()->generatedIconPixmap(mode, pm, &opt);

We could define a temporary custom QPalette for the disabled state in IconManager and set the original icon pixmap for normal state and our modified pixmap for disabled state.

3. Custom Application Style
As I said, Qt uses colors from the application palette. We can create a custom QStyle theme for Scribus and override the function for icon rendering. See https://doc.qt.io/qt-6/qcommonstyle.html#generatedIconPixmap

Option 2 looks like less effort for now. Option 3 has much more opportunities to have the application style under more control when Scribus runs on different operating systems.

The attached image shows the "normal" and "disabled" state under Adwaita dark theme.

medmedin2014

2024-01-05 19:12

reporter   ~0050833

@nitramr

>Perhaps the color contrast between "normal" and "disabled" is not much in Breeze theme

It's not specific to Breeze theme, because it happens with all 4 themes: Fusion, Windows, Oxygen and Breeze.

I'm running KDE Plasma 5.27.10 with official Dark Breeze theme, and all other Qt apps show clear difference between enabled and disabled icons.

cbradney

2024-01-05 19:26

administrator   ~0050834

@nitramr: With the proliferation of Wayland that seems to be accelerating, we may need to release a Qt6-based stable version this year, so 1.7.0 would move to stable 1.8.0. Any efforts really need to go into 1.7.0 now, and limited effort into 1.6.x that does not either stablise/fix it, or also contribute to 1.7.x. We certainly need to choose the most efficient fix for this that could be a part of 1.6.x series.

Luna Nightshade

2024-01-05 23:32

reporter   ~0050838

@cbradney I only changed the icons that could be disabled to have some yellow in it. It's possible to make them almost completely white to have even more contrast, though I also wouldn't make them completely yellow, as I don't think it would look any good.
If the solution with yellow highlights and a whiter tone of the icons work, I can easily change those – since they are all based on the 1_5_1 icon theme, there shouldn't be any license problems.

It would also make sense to change the corresponding light icon theme to match the dark one.

@nitramr From a design point of view I'd like a disabled icon theme. Then it would be possible to create "broken" icons, ones that look as if they're broken by a line pattern. But since this is not happening in 1.6.1.

Luna Nightshade

2024-01-14 18:50

reporter   ~0050892

Here's another version. If that's okay, I can look into making a corresponding light theme. That is editing the existing icons to have some color at the same locations as this one.
1_6_0_dark-3.zip (1,032,804 bytes)

Issue History

Date Modified Username Field Change
2019-11-28 12:54 medmedin2014 New Issue
2019-11-28 12:54 medmedin2014 File Added: Screenshot_20191128_134814.png
2019-11-28 13:30 jghali Severity major => minor
2020-11-02 18:28 Luna Nightshade Note Added: 0048297
2023-12-20 21:38 cbradney Note Added: 0050660
2024-01-02 21:27 medmedin2014 Note Added: 0050766
2024-01-02 21:27 medmedin2014 File Added: text1-2-9.png
2024-01-02 21:44 cbradney Note Added: 0050768
2024-01-02 21:44 cbradney File Added: image.png
2024-01-02 21:58 Luna Nightshade Note Added: 0050769
2024-01-02 22:14 Luna Nightshade Note Added: 0050771
2024-01-02 22:14 Luna Nightshade File Added: 1_6_0_dark.zip
2024-01-05 15:40 medmedin2014 Note Added: 0050822
2024-01-05 16:49 Luna Nightshade Note Added: 0050823
2024-01-05 16:49 Luna Nightshade File Added: 1_6_0_dark-2.zip
2024-01-05 16:51 Luna Nightshade Note Added: 0050824
2024-01-05 17:21 cbradney Note Added: 0050826
2024-01-05 17:26 medmedin2014 Note Added: 0050827
2024-01-05 17:26 medmedin2014 File Added: image1-6.png
2024-01-05 17:32 medmedin2014 Note Added: 0050828
2024-01-05 17:45 cbradney Note Added: 0050830
2024-01-05 18:59 nitramr Note Added: 0050832
2024-01-05 18:59 nitramr File Added: Bildschirmfoto vom 2024-01-05 19-59-00.png
2024-01-05 19:12 medmedin2014 Note Added: 0050833
2024-01-05 19:26 cbradney Note Added: 0050834
2024-01-05 23:32 Luna Nightshade Note Added: 0050838
2024-01-14 18:50 Luna Nightshade Note Added: 0050892
2024-01-14 18:50 Luna Nightshade File Added: 1_6_0_dark-3.zip