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

use the app menu bar for Linux instead of the default chrome #139

Merged
merged 1 commit into from
Apr 19, 2019

Conversation

shiftkey
Copy link
Owner

@shiftkey shiftkey commented Apr 7, 2019

Overview

Closes #114

Description

Currently when you run the app on Linux it uses the default chrome of the shell to render the menu and the minimize/maximize/close buttons:

This PR updates the Linux build to use the custom app menu we use on Windows, which should help with making the app feel more polished:

Release notes

Notes: [Improved] use styled app menu rather than native chrome on Linux

@j-f1
Copy link

j-f1 commented Apr 7, 2019

How does this interact with themes with custom title bar layouts?

@shiftkey
Copy link
Owner Author

shiftkey commented Apr 7, 2019

How does this interact with themes with custom title bar layouts?

Without examples it's hard to say specifically, but we disable the frame so this becomes more of a "chromeless" window, and makes it more consistent with how the app works on Windows.

@shiftkey shiftkey marked this pull request as ready for review April 19, 2019 13:14
@shiftkey shiftkey merged commit 4950957 into linux Apr 19, 2019
@shiftkey shiftkey deleted the app-menu-for-linux branch April 19, 2019 13:53
trxcllnt added a commit to trxcllnt/desktop that referenced this pull request Jul 1, 2019
trxcllnt added a commit to trxcllnt/desktop that referenced this pull request Jul 1, 2019
shiftkey added a commit that referenced this pull request Mar 28, 2020
shiftkey added a commit that referenced this pull request Apr 6, 2020
@Heath123
Copy link

I don't see the menuWhy was this reverted? Please at least make it an option...

@shiftkey
Copy link
Owner Author

@Heath123 please open a fresh issue and share some details about your setup (the version installed, what you expect to see, what is actually visible, screenshots, etc)

@Heath123
Copy link

@shiftkey This was reverted on purpose
8a5b751
OK though...

@shiftkey
Copy link
Owner Author

This was reverted on purpose

Yes, based on the feedback from #149

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.

menu bar in linux
3 participants