Regression: Dialogs with inputs can show an unwanted scrollbar when using large fonts |
|||||||||||
Issue descriptionVersion: 59.0.3071.47 OS: Ubuntu 14.04 Steps to reproduce the problem: 1. Launch chrome and go to chrome://settings, Change the font size to very large. 2.Go to content settings>>cookies>>Add a website under block/clear on exit/allow section 3. Click on 3 dot menu>>edit and observe What is the expected behavior? Scrollbar should not be displayed What went wrong? Instead unwanted scrollbar is displayed This is a regression issue broken in M59. Good Build: 59.0.3056.0 dev Bad Build: 59.0.3057.0 dev
,
May 10 2017
Issue is seen in Windows as well.
,
May 10 2017
Able to reproduce the issue on Mac 10.12.4 using chrome version 59.0.3071.47
,
May 11 2017
Using the per-revision bisect providing the bisect results, Good build:59.0.3056.0 - (Revision:460603). Bad build: 59.0.3057.0 - (Revision:460966). CHANGE-LOG URL: https://chromium.googlesource.com/chromium/src/+log/d94af97739e56d4174dc320b16d36f6795f2622f..5046f55745940bd248cb51c38d45e7ff39cc75f3 Review-Url: https://codereview.chromium.org/2783513002 dpapad@ Kindly take a look and please help us to reassign this issue to a right owner if not with respect to this change. Thanks.!
,
May 15 2017
+scottchen: I recall having talked about this before. Did we reach a resolution for "very large" font setting?
,
May 26 2017
Issue 726602 has been merged into this issue.
,
May 26 2017
Making the title more generic.
,
Jul 11 2017
,
Oct 24 2017
Marking bugs (mostly lower priority ones) that I am unlikely to get to soon as Available.
,
Jul 12
There is a chance that this is not happening anymore, now that our dialogs use cr-input instead of paper-input. Assigning to Scott to audit.
,
Jul 18
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/e3105ebb1b5c63e6b6aaf765be337ba35278a34f commit e3105ebb1b5c63e6b6aaf765be337ba35278a34f Author: Scott Chen <scottchen@chromium.org> Date: Wed Jul 18 01:23:15 2018 WebUI: paper-input clean-ups. This CL cleans up some remnant of paper-input, now that paper-input is mostly removed. Bug: 720276 Cq-Include-Trybots: luci.chromium.try:closure_compilation Change-Id: Id17298df731d1fbfc111fdb6db40202e1036dd4a Reviewed-on: https://chromium-review.googlesource.com/1139086 Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org> Commit-Queue: Scott Chen <scottchen@chromium.org> Cr-Commit-Position: refs/heads/master@{#575892} [modify] https://crrev.com/e3105ebb1b5c63e6b6aaf765be337ba35278a34f/chrome/test/data/webui/settings/a11y/edit_dictionary_a11y_test.js [modify] https://crrev.com/e3105ebb1b5c63e6b6aaf765be337ba35278a34f/chrome/test/data/webui/settings/a11y/manage_profile_a11y_test.js [modify] https://crrev.com/e3105ebb1b5c63e6b6aaf765be337ba35278a34f/ui/webui/resources/cr_elements/cr_dialog/cr_dialog.js [delete] https://crrev.com/31245ba491f26376881390519950fd553ac71e70/ui/webui/resources/cr_elements/paper_input_style_css.html [modify] https://crrev.com/e3105ebb1b5c63e6b6aaf765be337ba35278a34f/ui/webui/resources/cr_elements/shared_style_css.html [modify] https://crrev.com/e3105ebb1b5c63e6b6aaf765be337ba35278a34f/ui/webui/resources/cr_elements_resources.grdp
,
Jul 25
This issue should be obsolete since we switched to the new cr-input element. |
|||||||||||
►
Sign in to add a comment |
|||||||||||
Comment 1 by sc00335...@techmahindra.com
, May 10 2017