Chrome doesn’t collapse context menu separators if they are at the end of the context menu box
Reported by
zambonif...@gmail.com,
May 21 2016
|
||||||||
Issue descriptionIf you use the `contextMenus` API available to extensions and apps, Chrome collapses all context menu items of type “separator” into one. I believe that if a separator is at the end of a context menu box, it should disappear, but it currently doesn’t (50.0.2661.102 stable). I’ve attached both the `.crx` and the source of an app demonstrating this problem. You will, however, need to install from the `.crx` to actually see the problem.
,
May 23 2016
Hello. I cannot provide a screenshot because the context menu isn’t captured on screenshots (at least in my OS, Ubuntu 16.04). However, you did forget something: the context menu! You should right-click the blank window to see the problem. You may need to use the .CRX instead of the .ZIP, since chrome automatically adds extra items to the context menu for unpacked extensions.
,
May 24 2016
Thank you for providing more feedback. Adding requester "brajkumar@chromium.org" for another review and adding "Needs-Review" label for tracking. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
May 26 2016
@zambonifofex: Could you please have a look at the attached video and let us know if this is the correct procedure to repro this issue. Thank you.
,
May 26 2016
Are you sure you are using the CRX and not loading it from the source? “Reload”, “Simulate browser restart”, “Inspect” and “Inspect background page” are exactly the items I told Chrome adds for unpackaged apps. When these are there, you cannot see the problem.
,
May 26 2016
Here’s a video.
,
May 26 2016
Thank you for providing more feedback. Adding requester "rnimmagadda@chromium.org" for another review and adding "Needs-Review" label for tracking. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jun 1 2016
Able to repro this issue on Windows 7, MAC (10.11.5) & Ubuntu Trusty (14.04) for Google Chrome Canary Version - 53.0.2753.0 This is a Non-Regression issue existing since M46 - # 46.0.2455.0. Before this build, upon launching the provided app, the window just closes off.
,
Jul 9 2016
Moving this nonessential bug to the next milestone. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jul 22 2016
It would be nice to fix this, but marking as low priority since there is a pretty easy workaround. The code that controls this is ContextMenuMatcher::AppendExtensionItems in chrome/browser/extensions/context_menu_matcher.cc if anyone wants to have a look. Patches welcome!
,
Jul 4 2017
Tested this on Mac (10.12.5) and Ubuntu Trusty (14.04) for Chrome Stable 59.0.3071.115. See screenshots. This appears to be fixed. |
||||||||
►
Sign in to add a comment |
||||||||
Comment 1 by brajkumar@chromium.org
, May 23 2016Labels: Needs-Feedback