Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Dark Mode issues on Linux(via LAB) #5044

Merged
merged 7 commits into from
Apr 23, 2024

Conversation

jamincollins
Copy link
Contributor

@jamincollins jamincollins commented Apr 16, 2024

Description

This should address most (if not all) dark mode issues.

I've added routines based on LAB color space in an effort to ensure that foreground colors are not the same (or similar) to background colors.

https://en.wikipedia.org/wiki/CIELAB_color_space

I'm sure there are other areas and dialogs these changes fix. While there are some targeted changes that were needed, the core is a generalized solution that should just work for most areas.

Screenshots/Recordings/Graphs

Tabs

Before
image
image
image

After
image
image
image

Custom Gcode Editor

Before
image

After
image

Export Configs

Before
image

After
image

Network Tests

Before
image

After
image

Printer Settings X/Y Fields

Before
image
image

After
image
image

Rename Plate

Before
image

After
image

Send G-Code

Before
image

After
image

Unsaved changes

Before
image

After
image

Tests

Built and tested locally on Arch Linux. I did need to revert commit 1bb8fad due to instability it introduced, see #5030.

Signed-off-by: Jamin W. Collins <jamin.collins@gmail.com>
Signed-off-by: Jamin W. Collins <jamin.collins@gmail.com>
Signed-off-by: Jamin W. Collins <jamin.collins@gmail.com>
Signed-off-by: Jamin W. Collins <jamin.collins@gmail.com>
Signed-off-by: Jamin W. Collins <jamin.collins@gmail.com>
@jamincollins jamincollins mentioned this pull request Apr 16, 2024
11 tasks
@jamincollins
Copy link
Contributor Author

Found another area that is improved.

Before:
image

After:
image

@jamincollins
Copy link
Contributor Author

This also fixes a light mode issue #3936, and possibly others.

image

@discip
Copy link
Contributor

discip commented Apr 16, 2024

@jamincollins

First of all, thank you for your effort in fixing this issue once and for all! 😃👍

Unfortunately, not all of the fixes mentioned above work for me on my machine with the current build: https://github.com/SoftFever/OrcaSlicer/actions/runs/8708611679?pr=5044

Not working:

  • Custom Gcode Editor
  • Printer Settings X/Y Fields

Could not test:

  • Network Tests
  • Send G-Code

Everything else you mentioned above works. 👍

setup
OS CPU GPU RAM SSD
Manjaro (Gnome) Ryzen 9 7950X3D XFX 7900 XTX G.Skill Flare X5 32 GB Samsung 990 Pro 2 TB

@jamincollins
Copy link
Contributor Author

@discip what theme are you running?

I'll try this under Gnome here on Arch, and if it works I'll see about building a Manjaro test system.

Also, as indicated on this PR, I can't successfully run a build that includes commit 1bb8fad to run. Any build including it segfaults for me.

Signed-off-by: Jamin W. Collins <jamin.collins@gmail.com>
@discip
Copy link
Contributor

discip commented Apr 18, 2024

@jamincollins
Thank you for fixing this! 👍
Now the colors are correct, but unfortunately now there is another strange issue:

In Printer settings, if for example the Basic Information tab is open and I click in any text box to change a value, but then switch to another tab e.g.: Extruder, I'm not able to change any values ​​except those that were previously unreadable (e.g.: Extruder > Extruder offset > x: )!

However I'm not sure if that was present prior to your latest changes.

Off-topic:
There is another annoyance that's not directly related, but makes using the neat custom G-Code editing feature useless, as due to the size of the Edit Custom G-Code window, which can't be changed, the “save” button is hidden at the bottom of the screen.

😅 Absolutely off-topic:
Sorry, since the following is kind of abandoned I dare to ask: Could you by any chance have a look at this as well?:

https://github.com/SoftFever/OrcaSlicer/pull/2405

thanks in advance
kind regards

@jamincollins
Copy link
Contributor Author

In Printer settings, if for example the Basic Information tab is open and I click in any text box to change a value, but then switch to another tab e.g.: Extruder, I'm not able to change any values ​​except those that were previously unreadable (e.g.: Extruder > Extruder offset > x: )!

However I'm not sure if that was present prior to your latest changes.

I am not able to reproduce this on Arch or Manjaro.

Off-topic: There is another annoyance that's not directly related, but makes using the neat custom G-Code editing feature useless, as due to the size of the Edit Custom G-Code window, which can't be changed, the “save” button is hidden at the bottom of the screen.

Please create a separate enhancement request for this.

😅 Absolutely off-topic: Sorry, since the following is kind of abandoned I dare to ask: Could you by any chance have a look at this as well?:

https://github.com/SoftFever/OrcaSlicer/pull/2405

Please create a separate enhancement request for this.

@SoftFever
Copy link
Owner

Awesome!
Tested on Ubuntu and Windows, looks good

@SoftFever SoftFever changed the title Dark Mode Fixes (via LAB) Fix Dark Mode issues on Linux(via LAB) Apr 23, 2024
Copy link
Owner

@SoftFever SoftFever left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@SoftFever SoftFever merged commit 315eee3 into SoftFever:main Apr 23, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants