New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 769567 link

Starred by 1 user
Status: Fixed
Owner: ----
Closed: Oct 2017
Cc:
█ scottchen@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome , Mac
Pri: 2
Type: Bug



Sign in to add a comment

MD Extensions: Scroll position lost when exiting details view.

Project Member Reported by dpa...@chromium.org, Sep 28 2017 
Repro steps (see screencast):

1) Install enough extensions (and/or make window narrow) such that a scrollbar is shown.
2) scroll down to some extension
3) enter details view
4) exit details view

Expected: Scroll position is preserved.
Actual: The main page is scrolled back to the top.
scroll_position_lost.mp4
1.2 MB View Download
Project Member

Comment 1 by bugdroid1@chromium.org, Oct 3 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/4a0fd9070dbdf0ade4fde30e51a1ce31e072bfd7

commit 4a0fd9070dbdf0ade4fde30e51a1ce31e072bfd7
Author: dpapad <dpapad@chromium.org>
Date: Tue Oct 03 00:02:59 2017

MD Extensions: Create per-view scrollable containers.

Bug:  769568 ,  769567 
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I80710e77d7568acbfb28f402dea4bcd20c07d972
Reviewed-on: https://chromium-review.googlesource.com/691047
Commit-Queue: Demetrios Papadopoulos <dpapad@chromium.org>
Reviewed-by: Scott Chen <scottchen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#505862}
[modify] https://crrev.com/4a0fd9070dbdf0ade4fde30e51a1ce31e072bfd7/chrome/browser/resources/md_extensions/compiled_resources2.gyp
[modify] https://crrev.com/4a0fd9070dbdf0ade4fde30e51a1ce31e072bfd7/chrome/browser/resources/md_extensions/detail_view.html
[modify] https://crrev.com/4a0fd9070dbdf0ade4fde30e51a1ce31e072bfd7/chrome/browser/resources/md_extensions/detail_view.js
[modify] https://crrev.com/4a0fd9070dbdf0ade4fde30e51a1ce31e072bfd7/chrome/browser/resources/md_extensions/extensions.html
[modify] https://crrev.com/4a0fd9070dbdf0ade4fde30e51a1ce31e072bfd7/chrome/browser/resources/md_extensions/item_list.html
[modify] https://crrev.com/4a0fd9070dbdf0ade4fde30e51a1ce31e072bfd7/chrome/browser/resources/md_extensions/item_list.js
[modify] https://crrev.com/4a0fd9070dbdf0ade4fde30e51a1ce31e072bfd7/chrome/browser/resources/md_extensions/keyboard_shortcuts.html
[modify] https://crrev.com/4a0fd9070dbdf0ade4fde30e51a1ce31e072bfd7/chrome/browser/resources/md_extensions/keyboard_shortcuts.js
[modify] https://crrev.com/4a0fd9070dbdf0ade4fde30e51a1ce31e072bfd7/chrome/browser/resources/md_extensions/manager.html
[modify] https://crrev.com/4a0fd9070dbdf0ade4fde30e51a1ce31e072bfd7/chrome/browser/resources/md_extensions/manager.js
[modify] https://crrev.com/4a0fd9070dbdf0ade4fde30e51a1ce31e072bfd7/chrome/browser/resources/md_extensions/view_manager.html
[modify] https://crrev.com/4a0fd9070dbdf0ade4fde30e51a1ce31e072bfd7/chrome/test/data/webui/extensions/extension_keyboard_shortcuts_test.js

Comment 2 by dpa...@chromium.org, Oct 3 2017

Status: Fixed (was: Available)
►

Sign in to add a comment