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

Change context menu #1150

Closed
maximtop opened this issue Nov 22, 2018 · 2 comments
Assignees
Labels
Milestone

Comments

@maximtop
Copy link
Contributor

@maximtop maximtop commented Nov 22, 2018

Remove nested links to settings and use only one link to the settings
https://uploads.adguard.com/mtopciu_7lzm5.png

@maximtop maximtop added this to the 3.0 milestone Nov 22, 2018
@maximtop maximtop self-assigned this Nov 22, 2018
@ameshkov

This comment has been minimized.

Copy link
Member

@ameshkov ameshkov commented Jan 2, 2019

Too many menu items now:
https://uploads.adguard.com/up04_c00po_Monosnap.png

Here's how they should be sorted to get "check filters updates" back to the menu:
https://uploads.adguard.com/up04_a63l0_Example_Domain.png

@maximtop

This comment has been minimized.

Copy link
Contributor Author

@maximtop maximtop commented Jan 15, 2019

In Chrome <72 separators don't work
https://bugs.chromium.org/p/chromium/issues/detail?id=904220

Also, separators don't work in Chrome 73 if select some text or image before the context menu called

@zzebrum zzebrum closed this Feb 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.