Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

More bookmarks menu item no longer appears #7097

Closed
bbondy opened this issue Feb 6, 2017 · 1 comment
Closed

More bookmarks menu item no longer appears #7097

bbondy opened this issue Feb 6, 2017 · 1 comment
Assignees
@bbondy
Labels
QA/checked-macOS QA/checked-Win32 QA/checked-Win64 QA/test-plan-specified release-notes/include
Milestone
0.13.3

Comments

@bbondy
Copy link
Member

bbondy commented Feb 6, 2017

For some of the custom drawn menus we support dataL10nId but due to sanitization they are removed. Since we aren't calling the menu sanitization recursively it isn't seen very often, but it does make a valid menu item disappear in the bookmark menu at least, the last one that says more menu items.
@bbondy bbondy added this to the 0.13.3 milestone Feb 6, 2017
@bbondy bbondy self-assigned this Feb 6, 2017
@bbondy bbondy closed this as completed in 35944f6 Feb 6, 2017
@luixxiul
Copy link
Contributor

luixxiul commented Feb 6, 2017

Test plan: make sure "more bookmarks" menu item appears

This was referenced Feb 11, 2017
Closed
Closed
Closed
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@bbondy bbondy

Labels
QA/checked-macOS QA/checked-Win32 QA/checked-Win64 QA/test-plan-specified release-notes/include
Projects
None yet
Milestone
0.13.3
Development

No branches or pull requests
3 participants
@bbondy @alexwykoff @luixxiul