Skip to content

Conversation

@rhysd
Copy link
Contributor

@rhysd rhysd commented Aug 11, 2015

Hi.
Thank you for such a great app.

In OS X, gitify is now bound to specific work space where gitify firstly started. When I click the menu icon in other workspace, the window will be opened in the bound work space and current workspace is forced to be changed.

I used BrowserWindow.setVisibleOnAllWorkspaces() API to solve this issue.

https://github.com/atom/electron/blob/master/docs/api/browser-window.md#browserwindowsetvisibleonallworkspacesvisible

I think all menu bar applications should set this option to true.

In OS X, gitify is now bound to specific work space where gitify firstly
started.  When I click the menu icon in other workspace, the window will
be opened in the bound work space and current workspace is forced to be
changed.

I used `BrowserWindow.setVisibleOnAllWorkspaces()` API to solve this
issue.

https://github.com/atom/electron/blob/master/docs/api/browser-window.md#browserwindowsetvisibleonallworkspacesvisible

I think all menu bar application should set this option to true.
@manosim manosim added the enhancement New feature or enhancement to existing functionality label Aug 11, 2015
@rhysd
Copy link
Contributor Author

rhysd commented Aug 12, 2015

I think this PR can improve #20.

@manosim
Copy link
Member

manosim commented Aug 12, 2015

I've tried it before and I think it didn't work (maybe in older versions of Electron? who knows..) but now it works great! Thank you @rhysd! 🚀

manosim pushed a commit that referenced this pull request Aug 12, 2015
Make visible menu window on all workspaces
@manosim manosim merged commit db62bee into gitify-app:master Aug 12, 2015
@rhysd
Copy link
Contributor Author

rhysd commented Aug 13, 2015

✨🐶

@rhysd rhysd deleted the set-visible-on-all-workspaces branch August 13, 2015 17:08
@manosim manosim added this to the Release 0.0.15 milestone Aug 13, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or enhancement to existing functionality

Development

Successfully merging this pull request may close these issues.

2 participants