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

Some electron based applications use a white menu background color #715

Closed
otavioschwanck opened this issue Aug 14, 2018 · 22 comments
Closed

Comments

@otavioschwanck
Copy link

otavioschwanck commented Aug 14, 2018

image

Atom and Visual Studio Code have a white toolbat, making very ugly to look.

Other GTK 2 applications like franz, work nice:
image

But, when try to change theme to adwaita, it have a black toolbar, even when i restart the application.
image

@Feichtmeier
Copy link
Member

Hi thanks for reporting.

Both are based on electron, and are now using gtk3 =)
Just for the record!

Doesn't reduce the fact that they have the wrong color ofc :)

@Feichtmeier Feichtmeier changed the title GTK 2 Applications white toolbar Some electron based applications use a white menu background color Aug 14, 2018
@Paz-it
Copy link

Paz-it commented Aug 14, 2018

That's an anomaly I see when on communitheme snap with some applications,Gtk2 apps included. While trying to change theme some parts of communitheme are still present. It looks like a bug in snap session.
@otavioschwanck , are you seeing this issue using snap session?

@otavioschwanck
Copy link
Author

@Paz-it yes, using snap session. I'll try gnome session and Ubuntu session with XORG \ Wayland. Report here soon.

@otavioschwanck
Copy link
Author

otavioschwanck commented Aug 14, 2018

@Paz-it same results in all sessions.

New informations:

  • On Arc Darker Theme, same result. White Toolbar on Atom \ VSCode.
  • The ambiance theme is the only fine. Black toolbar on VSCODE \ Atom and Franz.

Ambiance:
image

Arc Darker:

image

i installed ubuntu 2 days ago.
In this two days, the VSCODE \ Atom bar are black on communittheme. Today i upgraded my system and the bar is white.

New information:

VS Code in ElementaryOS after upgrade, changed the toolbar to the inverse color, like in communittheme. Maybe is a electron issue?

@clobrano
Copy link
Member

Gtk2 apps included. While trying to change theme some parts of communitheme are still present

Have you tried rebooting Gnome shell? Those might be server side decorations that don't go away otherwise

@otavioschwanck
Copy link
Author

otavioschwanck commented Aug 14, 2018

@clobrano yes. Rebooted gnome shell in every change using "r". Also rebooted the system too. Problem persist.

@clobrano
Copy link
Member

@otavioschwanck Ok, @Paz-it you did the same?

@Feichtmeier
Copy link
Member

The menus are also white with the .deb version of atom.

But maybe we should first check if "they" really suck up the gtk code for the menus :)

@otavioschwanck
Copy link
Author

I'm using a deb version of atom, vscode and franz.

@Paz-it
Copy link

Paz-it commented Aug 14, 2018

@clobrano Yes, I did the same.

@Feichtmeier
Copy link
Member

I don't understand this...

Electron uses GTK3 dialogues now.
But is it possible that the menu is still styled with gtk2 styling? 🤔

@Eskander
Copy link

Same here. Using communitheme snap, vscode (deb) shows a white menu bar.

image

Not sure if it helps but the menu bar follows Adwaita-dark theme.

image

@clobrano
Copy link
Member

Both are based on electron, and are now using gtk3

@Feichtmeier when did this change happen? I remember electron apps where styled after gtk2

@Feichtmeier
Copy link
Member

@clobrano some weeks ago

@clobrano
Copy link
Member

Weird, because in #728 menu bar works like it's using GTK2 colors

@Feichtmeier
Copy link
Member

Feichtmeier commented Aug 19, 2018

Prbly because they didn't merge the newer electron code yet :D

But anyways I don't know why for example atom, which clearly uses the gtk3 filechoser , doesnt use our menu colour

@clobrano
Copy link
Member

@otavioschwanck

VS Code in ElementaryOS after upgrade, changed the toolbar to the inverse color, like in communittheme. Maybe is a electron issue?

not sure if I understood. Who changed the toolbar?

@clobrano
Copy link
Member

However, electron project opened a bug about this and there's a PR in progress, so I guess it's an upstream issue

electron/electron#13381

@Feichtmeier
Copy link
Member

I'd say let's close this here then? :) If it's handled upstream

@clobrano
Copy link
Member

clobrano commented Sep 26, 2018

The fix was merged electron/electron#14785

@Feichtmeier
Copy link
Member

@clobrano
Funny random coincidence, this would be fixed by the jet menus =)
https://discourse.ubuntu.com/t/mockups-new-design-discussions/1898/311

@Feichtmeier
Copy link
Member

Since the fix is in electron let's close it here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants