Regression: Vertical scroll bar is not seen for 'More actions' menu of chrome://downloads while resizing the browser window.
Reported by
jshan...@etouch.net,
Jun 15 2016
|
||||
Issue descriptionChrome Version: 53.0.2768.0 Revision 9cd2e77b728d6d5ac115acd944b37df70423c355-refs/heads/master@{#399800} OS: Windows(7,8,10),Mac(10.10.5,10.11.4),Linux(Ubuntu 14.04 LTS) Steps: 1. Launch chrome and download any page by selecting 'Save as' option from context menu. 2. Navigate to chrome://downloads, resize the browser window and click on 'More actions' menu. 3. Now keep on resizing the browser window towards L.H.S and observe 'More actions' menu. Actual: Vertical scroll bar is not seen for 'More actions' menu while resizing the browser. Expected: Vertical scroll bar should be seen for 'More actions' menu while resizing the browser. This is a regression issue broken in M-53, below is bisect info Good build: 53.0.2754.0 Bad build: 53.0.2755.0 Narrow bisect: https://chromium.googlesource.com/chromium/src/+log/851bbadbc2348dcf79851c9be1b766e4ce00a2ab..ee78dde42fe284e47ca51b1a5ba25b9b880ce43d?pretty=fuller&n=100 Suspecting: r397007 ? Please help to re-assign if your change is not the cause.
,
Jun 16 2016
My CL doesn't change anything related to the overflow menu. +dbeam to investigate
,
Jun 16 2016
yeahhhh, i wouldn't really say either of those screenshots are valid (there really should never be scollbars in a menu like that)
,
Jun 16 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/f04f35b8abb4c59b77e4e0898065df829a117e95 commit f04f35b8abb4c59b77e4e0898065df829a117e95 Author: dbeam <dbeam@chromium.org> Date: Thu Jun 16 22:34:15 2016 MD Downloads: hide ... (more options) menu on resize R=michaelpg@chromium.org BUG= 613927 , 620291 CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation Review-Url: https://codereview.chromium.org/2074483003 Cr-Commit-Position: refs/heads/master@{#400281} [modify] https://crrev.com/f04f35b8abb4c59b77e4e0898065df829a117e95/chrome/browser/resources/md_downloads/crisper.js [modify] https://crrev.com/f04f35b8abb4c59b77e4e0898065df829a117e95/chrome/browser/resources/md_downloads/toolbar.js [modify] https://crrev.com/f04f35b8abb4c59b77e4e0898065df829a117e95/chrome/browser/resources/md_downloads/vulcanized.html [modify] https://crrev.com/f04f35b8abb4c59b77e4e0898065df829a117e95/chrome/test/data/webui/md_downloads/downloads_browsertest.js [modify] https://crrev.com/f04f35b8abb4c59b77e4e0898065df829a117e95/chrome/test/data/webui/md_downloads/layout_tests.js [add] https://crrev.com/f04f35b8abb4c59b77e4e0898065df829a117e95/chrome/test/data/webui/md_downloads/toolbar_tests.js
,
Jun 16 2016
the ... menu now hides on resize, so we shouldn't be seeing these issues anymore :) |
||||
►
Sign in to add a comment |
||||
Comment 1 by tkonch...@chromium.org
, Jun 15 2016