Scroll jumps back to the top repeatedly when scrolled to see more bluetooth devices from settings page |
||||||||||||||||||
Issue descriptionVersion: 56.0.2924.7 / 9000.8.0 What steps will reproduce the problem? 1. Open Bluetooth sub menu from settings page 2. Click on "Add a device " 3. Scroll for the scanned devices What is the expected result? The scroll position should stay at the point the scroll stopped What happens instead? The scroll bar jumps back to the top Please use labels and text to provide additional information. Similar defect was fixed for the bluetooth scroll area in status tray. https://bugs.chromium.org/p/chromium/issues/detail?id=664930
,
Nov 29 2016
This code is webui, so it is unrelated to the system menu.
,
Nov 29 2016
,
Nov 29 2016
,
Nov 29 2016
Note: We should only care about this in the MD Settings UI. We have a similar problem with Internet settings. There are a few different approaches to this. Please cc me on any CL, I will be curious to see what you come up with.
,
Nov 29 2016
Also, this is a long standing issue in the old settings UI. It doesn't seem like a P1 / Release Blocker.
,
Dec 6 2016
No, this is not an issue with the previous UI. It is an annoying regression in behavior, so I'm adding back the blocker label. I can see on the same machine running M55 that the menu stays in the same position on the old settings page and jumps on the new one.
,
Dec 6 2016
Please clarify: Is this affecting the new MD Settings page (chrome://md-settings) or the old settings UI (chrome://settings-frame) ? If it is affecting the new UI, that is not shipping in 56, so this should not be a stable blocker.
,
Dec 6 2016
This is affecting the new MD Settings page (chrome://md-settings)
,
Dec 6 2016
OK. We should fix this for 57. MD Settings will not be the default on 56.
,
Jan 7 2017
MD Settings is not launching in M57 either, so moving milestone to M58.
,
Jan 11 2017
,
Jan 19 2017
Issue 665744 has been merged into this issue.
,
Jan 19 2017
Current propose solution is https://codereview.chromium.org/2627373002/
,
Jan 24 2017
,
Jan 24 2017
,
Feb 2 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/5c10efaf05bdde8188c305283ae207071674c0fd commit 5c10efaf05bdde8188c305283ae207071674c0fd Author: stevenjb <stevenjb@chromium.org> Date: Thu Feb 02 21:21:55 2017 MD Settings: Save and restore scroll position in iron-list TODO: cr-scrollable-behavior needs tests. BUG= 669248 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation Review-Url: https://codereview.chromium.org/2627373002 Cr-Commit-Position: refs/heads/master@{#447842} [modify] https://crrev.com/5c10efaf05bdde8188c305283ae207071674c0fd/chrome/browser/resources/settings/bluetooth_page/bluetooth_device_dialog.html [modify] https://crrev.com/5c10efaf05bdde8188c305283ae207071674c0fd/chrome/browser/resources/settings/bluetooth_page/bluetooth_device_dialog.js [modify] https://crrev.com/5c10efaf05bdde8188c305283ae207071674c0fd/chrome/browser/resources/settings/bluetooth_page/bluetooth_subpage.js [modify] https://crrev.com/5c10efaf05bdde8188c305283ae207071674c0fd/ui/webui/resources/cr_elements/cr_scrollable_behavior.html [modify] https://crrev.com/5c10efaf05bdde8188c305283ae207071674c0fd/ui/webui/resources/cr_elements/cr_scrollable_behavior.js [modify] https://crrev.com/5c10efaf05bdde8188c305283ae207071674c0fd/ui/webui/resources/cr_elements/network/cr_network_list.html [modify] https://crrev.com/5c10efaf05bdde8188c305283ae207071674c0fd/ui/webui/resources/cr_elements/network/cr_network_list.js
,
Feb 2 2017
,
Feb 4 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/e3cd1477cda2dc5affdd1b3711e507c6c2025fbe commit e3cd1477cda2dc5affdd1b3711e507c6c2025fbe Author: stevenjb <stevenjb@chromium.org> Date: Sat Feb 04 02:34:03 2017 Add cr_scrollable_behavior tests BUG= 669248 Review-Url: https://codereview.chromium.org/2670223002 Cr-Commit-Position: refs/heads/master@{#448150} [modify] https://crrev.com/e3cd1477cda2dc5affdd1b3711e507c6c2025fbe/chrome/test/data/webui/cr_elements/cr_elements_browsertest.js [add] https://crrev.com/e3cd1477cda2dc5affdd1b3711e507c6c2025fbe/chrome/test/data/webui/cr_elements/cr_scrollable_behavior_tests.js
,
Mar 30 2017
,
Mar 30 2017
Confirmed that this works during OOBE. Over to bluetooth test team to verify this bug
,
Mar 30 2017
works on BT menu in settings too. |
||||||||||||||||||
►
Sign in to add a comment |
||||||||||||||||||
Comment 1 by pbath...@chromium.org
, Nov 28 2016Labels: -Pri-2 M-56 ReleaseBlock-Stable Pri-1
Summary: Scroll jumps back to the top repeatedly when scrolled to see more bluetooth devices from settings page (was: Scroll jumps back to the top repeatedly when scrolled to see more devices from settings page)