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.

Sign up for GitHub

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

Jump to bottom

Keep menubar icon highlighted when popup is visible #27

Open
xhacker opened this issue Nov 10, 2014 · 3 comments
Open

Keep menubar icon highlighted when popup is visible #27

xhacker opened this issue Nov 10, 2014 · 3 comments
Labels
enhancement

Comments

@xhacker
Copy link
Member

xhacker commented Nov 10, 2014 •
edited
Loading

The highlight state should be kept after click on the icon.

Some clue: https://code.google.com/p/chromium/issues/detail?id=402787

--- Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/5884163-keep-menubar-icon-highlighted-when-popup-is-visible?utm_campaign=plugin&utm_content=tracker%2F899498&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F899498&utm_medium=issues&utm_source=github).
@luin
Copy link

luin commented Jul 16, 2015

+1 for this enhancement. The current behaviour seems to be a little strange IMO.

@xhacker xhacker self-assigned this Jul 20, 2015
@luin
Copy link

luin commented Jul 20, 2015

Awesome! Me-me-da!

xhacker added a commit that referenced this issue Sep 22, 2015
@xhacker
[Close #46] Revert "[Close #27] Keep icon highlight state after click."
7f7e123 
This reverts commit 5d28894.

The way to keep status item highlighted no longer works on El Capitan. Revert for now. Need a better approach.
@xhacker
Copy link
Member Author

xhacker commented Sep 22, 2015

Reopening because the approach no longer works on El Capitan.

@xhacker xhacker reopened this Sep 22, 2015
@xhacker xhacker changed the title Better menubar icon Keep menubar icon highlighted when popup is visible Sep 22, 2015
@xhacker xhacker removed their assignment Jul 3, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@luin @xhacker