Skip to content

Toggle tray between desktop mode and menubar mode #197

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

Closed

Conversation

br1anchen
Copy link

This PR fixes #196

I am totally new to Electron, so it might be not the best solution to this issue.
From doc, it seems like the only way to remove menubar icon is to call tray.destroy().
Then it would require getModeMenuItems() return ModeMenuItems always.

brunolemos and others added 30 commits July 2, 2019 19:12

Verified

This commit was signed with the committer’s verified signature. The key has expired.
brunolemos Bruno Lemos

Verified

This commit was signed with the committer’s verified signature. The key has expired.
brunolemos Bruno Lemos

Verified

This commit was signed with the committer’s verified signature. The key has expired.
brunolemos Bruno Lemos

Verified

This commit was signed with the committer’s verified signature. The key has expired.
brunolemos Bruno Lemos

Verified

This commit was signed with the committer’s verified signature. The key has expired.
brunolemos Bruno Lemos

Verified

This commit was signed with the committer’s verified signature. The key has expired.
brunolemos Bruno Lemos
This reverts commit b9f5ffa.

Verified

This commit was signed with the committer’s verified signature. The key has expired.
brunolemos Bruno Lemos
Not sure about this, might revert later

devhubapp#156

devhubapp#90

Verified

This commit was signed with the committer’s verified signature. The key has expired.
brunolemos Bruno Lemos

Verified

This commit was signed with the committer’s verified signature. The key has expired.
brunolemos Bruno Lemos
Could not reproduce on master version

Verified

This commit was signed with the committer’s verified signature. The key has expired.
brunolemos Bruno Lemos

Verified

This commit was signed with the committer’s verified signature. The key has expired.
brunolemos Bruno Lemos

Verified

This commit was signed with the committer’s verified signature. The key has expired.
brunolemos Bruno Lemos
…ions

devhubapp#126

Verified

This commit was signed with the committer’s verified signature. The key has expired.
brunolemos Bruno Lemos

Verified

This commit was signed with the committer’s verified signature. The key has expired.
brunolemos Bruno Lemos
…n hidden

devhubapp#126

Verified

This commit was signed with the committer’s verified signature. The key has expired.
brunolemos Bruno Lemos
…Interactions

Keeping hidden components rendered to avoid an annoying delay when opening the filters view

devhubapp#126

Verified

This commit was signed with the committer’s verified signature. The key has expired.
brunolemos Bruno Lemos
This will probably have some negative effect, e.g. problems on old android devices

devhubapp#126

devhubapp@448d096

devhubapp@f3ee8a1

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
* Fix Linux desktop app crash on boot (devhubapp#150).

Verified

This commit was signed with the committer’s verified signature. The key has expired.
brunolemos Bruno Lemos

Verified

This commit was signed with the committer’s verified signature. The key has expired.
brunolemos Bruno Lemos

Verified

This commit was signed with the committer’s verified signature. The key has expired.
brunolemos Bruno Lemos

Verified

This commit was signed with the committer’s verified signature. The key has expired.
brunolemos Bruno Lemos

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
an -> a

Verified

This commit was signed with the committer’s verified signature. The key has expired.
brunolemos Bruno Lemos
As seem at Concurrent Mode conference talk

Verified

This commit was signed with the committer’s verified signature. The key has expired.
brunolemos Bruno Lemos
Like app version and platform

Verified

This commit was signed with the committer’s verified signature. The key has expired.
brunolemos Bruno Lemos
In preparation for a new ColumnOptions component

Verified

This commit was signed with the committer’s verified signature. The key has expired.
brunolemos Bruno Lemos

Verified

This commit was signed with the committer’s verified signature. The key has expired.
brunolemos Bruno Lemos

Verified

This commit was signed with the committer’s verified signature. The key has expired.
brunolemos Bruno Lemos

Verified

This commit was signed with the committer’s verified signature. The key has expired.
brunolemos Bruno Lemos
brunolemos and others added 26 commits September 17, 2019 01:25
Bumps [axios](https://github.com/axios/axios) from 0.18.0 to 0.18.1.
- [Release notes](https://github.com/axios/axios/releases)
- [Changelog](https://github.com/axios/axios/blob/v0.18.1/CHANGELOG.md)
- [Commits](axios/axios@v0.18.0...v0.18.1)

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [lodash](https://github.com/lodash/lodash) from 4.17.11 to 4.17.13.
- [Release notes](https://github.com/lodash/lodash/releases)
- [Commits](lodash/lodash@4.17.11...4.17.13)

Signed-off-by: dependabot[bot] <support@github.com>
It now shows the wiki page title and the link shows the diff with what was edited
Fix deep link internal handling

Handle plan change
@brunolemos
Copy link
Member

Thanks for the PR! We shouldn’t remove the tray for everyone, since only one person complained about it. It could be an option where the user decides where they want to show (only tray, only dock or both).

@brunolemos
Copy link
Member

Hi, I removed the source code from GitHub and the pull request was automatically closed by GitHub. Sorry about that.

I don't know when I'll re-add the source code, but I'll re-open the pull request then.

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.

Option to remove menubar icon
4 participants