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

Insomnia dark theme should leverage dark theme on linux #2149

Open
nguyenkien opened this issue May 10, 2020 · 7 comments
Open

Insomnia dark theme should leverage dark theme on linux #2149

nguyenkien opened this issue May 10, 2020 · 7 comments
Labels
C-improvement Category: Improvement / Enhancement E-good-first-issue Experience required: Beginner / New N-discussion Needs: Discussion needs-triage O-linux Operating System: Linux

Comments

@nguyenkien
Copy link

When user choose dark theme title bar should use system dark theme too.

Screenshots
This is how it look now
Screenshot from 2020-05-10 20-17-16

This is how it should be.
Screenshot from 2020-05-10 20-22-05
This is command I use to change it everytime I open app:
xprop -f _GTK_THEME_VARIANT 8u -set _GTK_THEME_VARIANT "dark"

Desktop (please complete the following information):

  • OS: Linux
  • Installation Method: Github release page
  • App Version:
    Insomnia v7.1.1
    Designer 2020.1.2
@nguyenkien nguyenkien added the B-bug Bug: general classification label May 10, 2020
@nguyenkien nguyenkien changed the title Short issue title Insomnia dark theme May 10, 2020
@nijikokun nijikokun added Feature 🎁 N-investigation Needs: investigation and removed B-bug Bug: general classification labels May 12, 2020
@nijikokun
Copy link
Contributor

I'm not sure if this is possible through the electron API, but I'd love to be enlightened. It looks awesome.

@nijikokun nijikokun changed the title Insomnia dark theme Insomnia dark theme should leverage dark theme on linux May 12, 2020
@nguyenkien
Copy link
Author

nguyenkien commented May 12, 2020

Just foundout this on electron docs: Passing darkTheme: true when you create BrowserWindow
https://github.com/electron/electron/blob/v3.1.13/docs/api/browser-window.md

@nijikokun nijikokun added E-good-first-issue Experience required: Beginner / New and removed N-investigation Needs: investigation labels May 12, 2020
@plumcube
Copy link

May I pick up this issue?

@gschier
Copy link
Contributor

gschier commented May 14, 2020

I think we should discuss the implementation first. If we have to pass darkTheme: true when the app launches (when window is created), it won't be able to switch to dark when the user changes the theme.

So, a few questions:

  • How do we know if a theme is dark?
  • Is it possible to change the dark theme without relaunching the window?
  • Would this be better as a setting that the user could toggle separate from the theme?
  • I know IntelliJ used to have a setting for this on Mac. I see a vibrancy option in there that looks like it might do something similar on Mac.

@gschier gschier added N-discussion Needs: Discussion C-improvement Category: Improvement / Enhancement and removed Feature 🎁 labels May 14, 2020
@nguyenkien
Copy link
Author

For vscode, theme have to identify them self is dark or light. You could do the same
Screenshot from 2020-05-15 22-02-40

@plumcube
Copy link

plumcube commented May 15, 2020 via email

@stale
Copy link

stale bot commented May 31, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label May 31, 2021
@filfreire filfreire added the O-linux Operating System: Linux label Aug 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-improvement Category: Improvement / Enhancement E-good-first-issue Experience required: Beginner / New N-discussion Needs: Discussion needs-triage O-linux Operating System: Linux
Projects
None yet
Development

No branches or pull requests

6 participants