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

menu bar in linux #114

Closed
kerolloz opened this issue Jan 5, 2019 · 7 comments · Fixed by #139
Closed

menu bar in linux #114

kerolloz opened this issue Jan 5, 2019 · 7 comments · Fixed by #139
Assignees
Labels

Comments

@kerolloz
Copy link

kerolloz commented Jan 5, 2019

Description

Version

  • GitHub Desktop: GitHubDesktop-linux-1.5.1-linux2.deb
  • Operating system: Ubuntu 18.10

Steps to Reproduce

Expected Behavior

menu bar doesn't seem to be working properly

Actual Behavior

menu bar should appear with the names for each menu

Additional Information

image

Logs

I have no idea how to access logs in linux!

@vitalkanev
Copy link

I believe this is an issue with the Yaru theme. See ubuntu/yaru#969

@kerolloz
Copy link
Author

kerolloz commented Jan 6, 2019

@vitalkanev
Okay this solves the issue when I change the theme to Yaru Dark. but I really like the light one.

@shiftkey
Copy link
Owner

shiftkey commented Jan 6, 2019

@kerolloz thanks for the feedback. I actually think switching to use the in-app menu that Windows supports here is the best thing to do here.

Here's an example of it in action:

With this change we should make it a chromeless window, but maybe that's not the best idea.

@kerolloz
Copy link
Author

kerolloz commented Jan 8, 2019

@shiftkey I'm sorry. I didn't get what Windows has to do with my Ubuntu.

we should make it a chromeless window

How would that help me fix my problem?

@shiftkey
Copy link
Owner

shiftkey commented Jan 8, 2019

@kerolloz it would address the problem by not using the OS default menu (it would also blend in nicer on OSes with light themes)

@kerolloz
Copy link
Author

kerolloz commented Jan 8, 2019

@shiftkey Thanks, I got it.
Does that have to do with Electron(or its version)? :electron:
cause I got the same issue with Atom! :atom:
but surprisingly Slack is working properly 😃.

@shiftkey
Copy link
Owner

shiftkey commented Jan 8, 2019

Does that have to do with Electron(or its version)?

It's related to Electron and it trying to show a default menu that fits with the OS theme. electron/electron#15878 seems to be the fix but I think it's only just made it into 4.0.1 - maybe it'll be backported to earlier versions in time.

but surprisingly Slack is working properly 😃

I'm not sure what Slack is doing here - I haven't tested it out on Linux.

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

Successfully merging a pull request may close this issue.

3 participants