[MD]Menu list stays even after clicking anywhere on header in chrome://history
Reported by
jshan...@etouch.net,
Apr 14 2016
|
|||||||||||
Issue descriptionChrome Version: 51.0.2704.7 a8bebc2b0cd3e3018b7c9f6ac698f04dd226abba-refs/branch-heads/2704@{#48} 32/64 bit OS: All (Win 7 Aero-Enabled) Pre-condition:Enable Material Design history from chrome://flags Steps: 1. Launch Chrome and go to chrome://history. 2. Click on More menu icon (three dots), then click anywhere on Header and observe. Actual: Highlight disappears but list box of Menu stays open even after clicking anywhere on Header Expected: List box of Menu should not stay open after clicking anywhere on Header. This is a non-regression issue seen from 'M-50', build no-50.0.2638.0 ChangeLog URL: https://chromium.googlesource.com/chromium/src/+log/50.0.2637.0..50.0.2638.0?pretty=fuller&n=10000 Suspecting: r372589 ? Note: This is a new feature introduce in M-50
,
Jun 1 2016
Moving this nonessential bug to the next milestone. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jun 21 2016
,
Jun 22 2016
,
Jul 13 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/9aecdea1b2386028f8b3ad5e47fd6d98a6add2d9 commit 9aecdea1b2386028f8b3ad5e47fd6d98a6add2d9 Author: tsergeant <tsergeant@chromium.org> Date: Wed Jul 13 05:55:57 2016 MD WebUI: Reimplement cr-shared-menu using iron-dropdown. This change brings the implementation much closer to that of paper-menu-button, and fixes a number of longstanding bugs. In particular, * Focus is correctly restored to the element which opened the menu after the menu is closed. * It is only necessary to press escape once to close the menu. * It is possible to close the menu by clicking anywhere else on the page. * Tab focusing is trapped inside the menu, rather than allowing tab to go to a different part of the page. * Bold text is no longer applied the menu item which was last clicked. The main trade-off of this approach is that we now have to use neon-animation to animate the menu out, and due to the way that the menu is positioned it will animate in from the top-left corner (the same as paper-menu-button). BUG= 589362 , 603454 , 616074 , 619839 , 620038 CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation Review-Url: https://codereview.chromium.org/2104013004 Cr-Commit-Position: refs/heads/master@{#405048} [modify] https://crrev.com/9aecdea1b2386028f8b3ad5e47fd6d98a6add2d9/chrome/test/data/webui/cr_elements/cr_elements_browsertest.js [add] https://crrev.com/9aecdea1b2386028f8b3ad5e47fd6d98a6add2d9/chrome/test/data/webui/cr_elements/cr_shared_menu_tests.js [modify] https://crrev.com/9aecdea1b2386028f8b3ad5e47fd6d98a6add2d9/chrome/test/data/webui/md_history/history_overflow_menu_test.js [modify] https://crrev.com/9aecdea1b2386028f8b3ad5e47fd6d98a6add2d9/ui/webui/resources/cr_elements/cr_shared_menu/compiled_resources2.gyp [modify] https://crrev.com/9aecdea1b2386028f8b3ad5e47fd6d98a6add2d9/ui/webui/resources/cr_elements/cr_shared_menu/cr_shared_menu.html [modify] https://crrev.com/9aecdea1b2386028f8b3ad5e47fd6d98a6add2d9/ui/webui/resources/cr_elements/cr_shared_menu/cr_shared_menu.js
,
Jul 13 2016
This issue has been moved once and is lower than Pri-1. Removing the milestone. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jul 13 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/9aecdea1b2386028f8b3ad5e47fd6d98a6add2d9 commit 9aecdea1b2386028f8b3ad5e47fd6d98a6add2d9 Author: tsergeant <tsergeant@chromium.org> Date: Wed Jul 13 05:55:57 2016 MD WebUI: Reimplement cr-shared-menu using iron-dropdown. This change brings the implementation much closer to that of paper-menu-button, and fixes a number of longstanding bugs. In particular, * Focus is correctly restored to the element which opened the menu after the menu is closed. * It is only necessary to press escape once to close the menu. * It is possible to close the menu by clicking anywhere else on the page. * Tab focusing is trapped inside the menu, rather than allowing tab to go to a different part of the page. * Bold text is no longer applied the menu item which was last clicked. The main trade-off of this approach is that we now have to use neon-animation to animate the menu out, and due to the way that the menu is positioned it will animate in from the top-left corner (the same as paper-menu-button). BUG= 589362 , 603454 , 616074 , 619839 , 620038 CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation Review-Url: https://codereview.chromium.org/2104013004 Cr-Commit-Position: refs/heads/master@{#405048} [modify] https://crrev.com/9aecdea1b2386028f8b3ad5e47fd6d98a6add2d9/chrome/test/data/webui/cr_elements/cr_elements_browsertest.js [add] https://crrev.com/9aecdea1b2386028f8b3ad5e47fd6d98a6add2d9/chrome/test/data/webui/cr_elements/cr_shared_menu_tests.js [modify] https://crrev.com/9aecdea1b2386028f8b3ad5e47fd6d98a6add2d9/chrome/test/data/webui/md_history/history_overflow_menu_test.js [modify] https://crrev.com/9aecdea1b2386028f8b3ad5e47fd6d98a6add2d9/ui/webui/resources/cr_elements/cr_shared_menu/compiled_resources2.gyp [modify] https://crrev.com/9aecdea1b2386028f8b3ad5e47fd6d98a6add2d9/ui/webui/resources/cr_elements/cr_shared_menu/cr_shared_menu.html [modify] https://crrev.com/9aecdea1b2386028f8b3ad5e47fd6d98a6add2d9/ui/webui/resources/cr_elements/cr_shared_menu/cr_shared_menu.js
,
Jul 14 2016
The menu now closes correctly when clicking the toolbar. |
|||||||||||
►
Sign in to add a comment |
|||||||||||
Comment 1 by nyerramilli@chromium.org
, Apr 14 2016