Regression: Spacing is not proper in downloads page when font size is changed |
||||||||
Issue descriptionChrome Version: 61.0.3148.0 dev OS: Ubuntu 14.04 , Windows What steps will reproduce the problem? (1)Launch chrome and go to chrome://downloads[make sure you have some downloaded content] (2)Now go to chrome://settings and change font size to very large , navigate back to downloads page and observe top downloaded item and spacing between them (3)Now in settings page again change font size to medium and observe spacing between items , refresh page and observe spacing. Expected: After changing font no such overlapping should be seen and spacing should be proper. Actual: Instead first item is overlapped with second and only after refresh correct spacing is seen [on changing font size] This is a regression issue broken in M48. Good Build: 48.0.2536.0 dev Bad Build: 48.0.2537.0 dev
,
Jul 4 2017
CHANGELOG URL: https://chromium.googlesource.com/chromium/src/+log/e4471ed4dce8f2fc53e3e7f54aa8c4e18b93ea8c..ced9579f2d69e7d3379a31c3c977339dbf7bfa22 Suspecting https://codereview.chromium.org/1375333004 from changelog @dbeam: Please confirm the behavior and help in re-assigning if it is not related to your change.
,
Jul 6 2017
I'm no longer working on Chrome, and unlikely to fix any bug I'm currently assigned. So this bug doesn't languish, I'm unassigning myself.
,
Jul 6 2017
,
Jul 7 2017
@dbeam: Do you know if there is any "resize" event fired when the default font size is change via settings? The problem could simply be fixed by instructing the iron-list to resize itself, but need a signal to hook up to from the Downloads page.
,
Jul 11 2017
there's no web-available signal for font-size (or zoom) change that I'm aware of. detecting zoom and font-size _changes_ on the web is a fairly long-lived, annoying problem, imo. here's hoping there's something magical I just haven't heard about. fwiw: a quick google revealed that you can fairly easily poll web-available state, but that's kinda ghetto/drains batteries: https://alistapart.com/article/fontresizing https://alistapart.com/d/fontresizing/textresizedetector.js I tried this kinda lame way new MutationObserver(...).observe(document.documentElement, {attributes: true, attributeFilter: ['clientWidth']}) on jsfiddle with no success (but didn't try suppper hard). because this is a fancy chrome:// UI, you could certainly observe the inputs to the font-size or zoom (i.e. prefs or something) on the browser side and notify via FireWebUIListener[1]. [1] https://chromium.googlesource.com/chromium/src/+/master/docs/webui_explainer.md#WebUIMessageHandler_FireWebUIListener
,
Oct 24 2017
Marking bugs (mostly lower priority ones) that I am unlikely to get to soon as Available.
,
Oct 25
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue. Sorry for the inconvenience if the bug really should have been left as Available. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot |
||||||||
►
Sign in to add a comment |
||||||||
Comment 1 by krajshree@chromium.org
, Jul 4 2017Status: Untriaged (was: Unconfirmed)