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

Dark title bar when dark mode enabled #2125

Closed
3 tasks done
balazscseh opened this issue Jul 2, 2023 · 3 comments
Closed
3 tasks done

Dark title bar when dark mode enabled #2125

balazscseh opened this issue Jul 2, 2023 · 3 comments
Assignees
Labels
conclusion: duplicate Has already been submitted topic: code Related to content of the project itself type: enhancement Proposed improvement

Comments

@balazscseh
Copy link

balazscseh commented Jul 2, 2023

Describe the request

I want the title bar to be white when i use light theme of the app, and be dark when i use dark theme.

Also it would be great if there would be an option to change this automatically, based on system settings.

https://learn.microsoft.com/en-us/windows/apps/desktop/modernize/apply-windows-themes#enable-a-dark-mode-title-bar-for-win32-applications

Describe the current behavior

No matter if i enable dark mode in Windows settings, or in the app, the title bar remains white.

titlebarwhite

Arduino IDE version

2.1.1

Operating system

Windows

Operating system version

11

Additional context

I found almost the same previous request in the issue tracker BUT i also found eclipse-theia/theia#10044, if i understand correctly, it is possible to change the title bar to dark with custom settings.

Issue checklist

  • I searched for previous requests in the issue tracker
  • I verified the feature was still missing when using the latest nightly build
  • My request contains all necessary details
@balazscseh balazscseh added the type: enhancement Proposed improvement label Jul 2, 2023
@per1234
Copy link
Contributor

per1234 commented Jul 2, 2023

I found almost the same previous request in the issue tracker

Are you referring to this issue?: #321

I see that was closed as fixed by #2027. I'm not sure what the relevant change was in that PR though. I see that eclipse-theia/theia#10044 was shipped in Theia 1.19.0, so already part of the Theia version 1.31.1 that was in use by Arduino IDE prior to the bump in #2027 so it wasn't that the bump pulled in the support added in eclipse-theia/theia#10044.

I believe the idea is that the issue can be resolved by the removal of the Windows native title bar entirely by changing the window.titleBarStyle advanced setting to custom. The IDE's theming should then apply to the custom title bar. However, that setting doesn't work currently (#1733). I see there is work in progress to make it functional: #1870

@kittaakos
Copy link
Contributor

I'm not sure what the relevant change was in that PR though.

Updating the electron version was the fix. See here: https://github.com/arduino/arduino-ide/pull/2027/files#diff-60e7eeb6bc8be48e5693ebc9be980d76bb7973ab861bf167c666b519d437f501R26.

If you get the nightly, this is how it works: https://www.arduino.cc/en/software#nightly-builds.

Also it would be great if there would be an option to change this automatically, based on system settings.

It works like you asked for. However, IDE2 needs a restart to get the OS theme into the IDE2 theme.

#1870 is just icing on supporting the custom title bar beside the native one.

If you're wondering why this feature is unavailable on 2.2.1 but in the nightly build, please see #2119 (comment).

I am closing at resolved.

@kittaakos kittaakos added the conclusion: resolved Issue was resolved label Jul 3, 2023
@kittaakos kittaakos self-assigned this Jul 3, 2023
@kittaakos kittaakos added the topic: code Related to content of the project itself label Jul 3, 2023
@balazscseh
Copy link
Author

Thanks guys!

@per1234 per1234 added conclusion: duplicate Has already been submitted and removed conclusion: resolved Issue was resolved labels Aug 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
conclusion: duplicate Has already been submitted topic: code Related to content of the project itself type: enhancement Proposed improvement
Projects
None yet
Development

No branches or pull requests

3 participants