gama.old icon indicating copy to clipboard operation
gama.old copied to clipboard

Inactive greyed icons incorrectly displayed on Linux

Open AlexisDrogoul opened this issue 3 years ago • 7 comments

Steps to reproduce

  1. Run any model

  2. Observe the "step" icon (second from the left)

  3. When the simulation is run, it becomes gray and blurry (as if its size is doubled).

Capture d’écran du 2022-07-11 12-13-24

It happens with other icons as well (back/forward icons in the editor when no text has been entered, "Simulation Perspective Icon" when no experiment is running, etc...

It seems to be related to the fact that I am running on an HiDPI screen. Can someone test this on "normal" screens ?

System and version

Gama git version, Ubuntu 20.04

AlexisDrogoul avatar Jul 11 '22 05:07 AlexisDrogoul

Tried on my laptop: no problem as long as I keep a 100% scaling, if I change to any other value I get the same problem

lesquoyb avatar Jul 13 '22 08:07 lesquoyb

I tried the new bug fix, nothing changed on my computer. In case it can help: I noticed that the icon size is smaller with a scaling of 200%: image than with a scaling of 125%: image which is the opposite of what I would have expected

lesquoyb avatar Aug 08 '22 03:08 lesquoyb

The last commit effectively does not solve the issues in the global toolbar, which is described in the plugin.xml (see the screenshot) and not programmatically -- so all the changes are managed by SWT directly.

Image 2022-08-08 17-02-32

This issue is maybe related to this : https://bugs.eclipse.org/bugs/show_bug.cgi?id=568407, which is however marked as solved. Does changing GTK version affect the outcome ?

The positive point, however, is that the fix seems to solve the editors' toolbars icons, which are managed programmatically in GAMA: before the fix, the "greyed out/disabled" icons where computed automatically by SWT (like in the global toolbar), now they are computed in GamaIcons, indicating that SWT does not take into account the "disabled" icon in the plugin.xml and that the problem is in this "disablement" computation on Linux. This is confirmed by https://bugs.eclipse.org/bugs/show_bug.cgi?id=575492 that pinpoints exactly the same issue (w/o a fix or a workaround, however).

AlexisDrogoul avatar Aug 08 '22 10:08 AlexisDrogoul

Should be tested with the new version of Eclipse (either 2022-06 - https://www.eclipse.org/downloads/packages/release/2022-06/r/eclipse-ide-java-and-dsl-developers - or the upcoming 2022-09 - https://download.eclipse.org/eclipse/downloads/drops4/S-4.25M3-202208171800/ -)

AlexisDrogoul avatar Aug 21 '22 10:08 AlexisDrogoul

@AlexisDrogoul I'm building some GAMA with Eclipse 2022-06, I let you try to see if it fix the issue and doesn't create any new ones : https://github.com/RoiArthurB/GAMA-182-test-builder/actions/runs/2903093404

PS: On my Linux I never saw this issue and can't reproduce it :/

RoiArthurB avatar Aug 22 '22 10:08 RoiArthurB

Problem has been fixed on the SWT side (https://github.com/eclipse-platform/eclipse.platform.swt/pull/327), but it won't be out for 4.25 (Eclipse 2022-09). Should we wait til December ?

AlexisDrogoul avatar Sep 09 '22 02:09 AlexisDrogoul

I don't think waiting to try with the next release would be an issue considering that there will still be some work going on until at least October (and the release could be out before December like it was the case for the September release)

lesquoyb avatar Sep 09 '22 03:09 lesquoyb

We should build GAMA with 2022-12 now that #3553 is fixed -- and this would remove this issue as well.

AlexisDrogoul avatar Dec 18 '22 09:12 AlexisDrogoul

Fixed with the issue #3553 and the merge of #3544 :)

RoiArthurB avatar Dec 19 '22 03:12 RoiArthurB