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

on Linux default to "showHamburgerMenu": true #2666

Merged
merged 1 commit into from
Mar 17, 2018

Conversation

jcrben
Copy link
Contributor

@jcrben jcrben commented Feb 10, 2018

I believe this is ready to merge, but I didn't go through and run from source to check. This is partly to open a discussion and get a better understanding of why this default was selected. Another possible resolution to the underlying issue is figuring out how to get the title menu bar to work consistently on Linux.

If a minimal flow is felt to be super important, then I recommend an in-app onboarding wizard to help users understand how to access their menu bar. But until that's built, a menu should be visible by default.

From the code comment // show by default on Windows and Linux, it appears that the author expected the window menu to appear on Linux, but this isn't actually happening. Therefore I think it would be better to default to showing the hamburger menu and letting experienced users configure it away.

When I loaded up hyper on Linux, I couldn't access the menu at all. I saw this - and I didn't have an overall menu bar like macOS on Ubuntu 17.10 running KDE Plasma thru Virtualbox:
image

Issues from this that I found quickly:

Blame points to #946 which does not discuss this issue.

There is a mention of the frameless thing which I haven't messed around with (#795 - maybe that could help with #1683 and showing the title menu bar).

@chabou
Copy link
Contributor

chabou commented Feb 25, 2018

I am agreed. We need to enable it by default on Linux.
Thank you so much for your PR.

Do you mind to modify comment in default config file too?
https://github.com/zeit/hyper/blob/76beaa32f5f3285bc8f7083fff8cd722ba97f60a/app/config/config-default.js#L56

@chabou chabou added this to the 2.1.0 milestone Feb 25, 2018
Copy link
Contributor

@chabou chabou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Config comment should be updated.

@jcrben jcrben force-pushed the change-showHamburgerMenu-default branch from 9a0a261 to 79ceac2 Compare March 4, 2018 18:19
@jcrben
Copy link
Contributor Author

jcrben commented Mar 4, 2018

Updated. I think this gets into the whole Client Side Decorations going on in the Linux world. GNOME has an initiative to focus on client-side window decorations (https://wiki.gnome.org/Initiatives/CSD).

Using the hamburgerMenu aligns hyper with GNOME, but also I'm running KDE on Ubuntu 17.10 and it also isn't showing native menus, so it seems to cover modern KDE too (granted, I'm building Ubuntu from a server ISO https://gitlab.com/jcrben-play-learn/packer-desktop). I think that frameless param mentioned above has something to do with no native menus.

@jcrben jcrben force-pushed the change-showHamburgerMenu-default branch from 79ceac2 to 7c753d1 Compare March 4, 2018 18:25
@chabou chabou merged commit 7838a10 into vercel:canary Mar 17, 2018
@jcrben jcrben deleted the change-showHamburgerMenu-default branch March 18, 2018 01:13
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.

2 participants