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

There is unexpected "Show Tab Bar" menu item on macOS 10.12 #337

Closed
4 tasks done
hamishgraham opened this issue Oct 6, 2016 · 5 comments
Closed
4 tasks done

There is unexpected "Show Tab Bar" menu item on macOS 10.12 #337

hamishgraham opened this issue Oct 6, 2016 · 5 comments

Comments

@hamishgraham
Copy link

I confirm (by marking "x" in the [ ] below):


Summary

The Show Tab Bar menu item is inconsistent with team tabs.

Steps to reproduce

Environment

  • OS X 10.12
  • Mattermost Desktop App version: 3.4.1
  • Mattermost Server version: v3.3.0

On OS X there is a menu item called Show Tab Bar. This menu item show/hides what appears to be an Electron provided tab bar that is different to the team tabs. See screenshot: http://imgur.com/a/DBMXC

Expected behavior

The Show Tab Bar menu item should show/hide the Team tabs.

Observed behavior

The Show Tab Bar menu item shows a different additional tab bar.

Possible fixes

Option1: Hide the Show Tab Bar menu item.

Option2: Make the Show Tab Bar menu item show/hide the team tabs.

Option3: Adopt the provided tab functionality instead of the existing team tabs. This allows the tabs to be consistent with other applications and allows for behaviour like dragging tabs to a new window, combining tabs from windows etc.

@yuya-oc
Copy link
Contributor

yuya-oc commented Oct 6, 2016

I can't see the menu on my macOS 10.12.

no-show-tab-bar

But probably it's a issue which was fixed at electron v1.4.2. electron/electron#7415

@yuya-oc yuya-oc changed the title Show Tab Bar menu item does not work with team tabs There is unexpected "Show Tab Bar" menu item on macOS 10.12 Oct 7, 2016
@yuya-oc yuya-oc added this to the v3.5.0 milestone Oct 7, 2016
@yuya-oc
Copy link
Contributor

yuya-oc commented Oct 7, 2016

@hamishgraham This should be solved by using electron v1.4.2 (#307). Would you try the latest build?
https://circleci.com/gh/mattermost/desktop/734#artifacts

@hamishgraham
Copy link
Author

Sure no problem - I can confirm that the above build does not show the extra "Show Tab Bar" menu item.

@yuya-oc
Copy link
Contributor

yuya-oc commented Oct 10, 2016

Thanks for your confirmation!

@yuya-oc yuya-oc closed this as completed Oct 10, 2016
yuya-oc added a commit that referenced this issue Oct 10, 2016
@sloantash
Copy link

I'm having this issue suddenly in MacOS 10.14.3 with electron 3.1.6. Any ideas? I've got a view menu with the only option being role: "togglefullscreen", and that works, but those two items appear above it.

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

3 participants