New issue
Advanced search Search tips

Issue 754697 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jan 2018
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Feature



Sign in to add a comment

Keep scroll position in Console when clicking on stack trace source

Reported by kirill.m...@gmail.com, Aug 11 2017

Issue description

Chrome Version       : 60.0.3112.90
OS Version: OS X 10.12.6
URLs (if applicable) :
Other browsers tested:
  Add OK or FAIL after other browsers where you have tested this issue:
     Safari 5: OK
  Firefox 4.x: OK
     IE 7/8/9:

What steps will reproduce the problem?
1. Open a page with a javascript error on it with stacktrace and many messages after it
2. Scroll to stacktrace and click on a stacktrace line to open code in Sources panel
3. Click on a Console tab 

What is the expected result?

  The Console should open on the same scrolling position which was before clicking on stacktrace item.

What happens instead of that?

  The Console shows the last line of the Console

Please provide any additional information below. Attach a screenshot if
possible.

  If I right-click on the link and open it using "Open in Sources panel" item, clicking to Console tab back works as expected, i.e. the position is preserved.

UserAgentString: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/60.0.3112.90 Safari/537.36



 
Components: Platform>DevTools Platform>DevTools>UX
Labels: -Type-Bug -OS-Mac OS-All Type-Feature
Status: Untriaged (was: Unconfirmed)
Summary: Keep scroll position in Console when clicking on stack trace source (was: Clicking on a stacktrace line in Chrome dev tools and clicking back on Console tab does not keep scrolling position)
Owner: l...@chromium.org
Status: Assigned (was: Untriaged)

Comment 4 by l...@chromium.org, Aug 11 2017

Labels: Hotlist-Polish
Thanks for the report.  I can reproduce, it's weird that right click > "Open in sources" will preserve scroll position while regular click on a link does not.  In theory they should both be detaching the console panel, but I need to investigate.

In the meantime, another workaround that might be easier than right clicking is to have the drawer open with console (pressing Esc in DevTools), to see both console and sources at the same time.
Hello, thanks for the hint with the open drawer, but I often work on a notebook and keeping both sources and console is not very practical - I see too little on the screen at the same time.

And thanks a lot for looking into this, hope the fix won't be difficult :)


Project Member

Comment 6 by bugdroid1@chromium.org, Dec 20 2017

Project Member

Comment 7 by bugdroid1@chromium.org, Jan 17 2018

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

commit 28f1a1eacc3d88986bf5b92d1acdc17983891834
Author: Erik Luo <luoe@chromium.org>
Date: Wed Jan 17 04:50:02 2018

DevTools: preserve Console scrollTop when opening drawer

When opening the Console drawer, the Widget system tries to
restore scrollTop. However, it fails when the message
viewport has no scrollbar.

This CL bounds the max-height of the searchable view, so
that Console scroll can be restored.

Bug:  754697 
Change-Id: Id4a971bc5e9e4e2549015e3c8fe7e31b074c476d
Reviewed-on: https://chromium-review.googlesource.com/833501
Commit-Queue: Erik Luo <luoe@chromium.org>
Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#529613}
[modify] https://crrev.com/28f1a1eacc3d88986bf5b92d1acdc17983891834/third_party/WebKit/LayoutTests/http/tests/devtools/console/console-preserve-scroll-expected.txt
[modify] https://crrev.com/28f1a1eacc3d88986bf5b92d1acdc17983891834/third_party/WebKit/LayoutTests/http/tests/devtools/console/console-preserve-scroll.js
[modify] https://crrev.com/28f1a1eacc3d88986bf5b92d1acdc17983891834/third_party/WebKit/Source/devtools/front_end/console/ConsoleView.js
[modify] https://crrev.com/28f1a1eacc3d88986bf5b92d1acdc17983891834/third_party/WebKit/Source/devtools/front_end/console/consoleView.css
[modify] https://crrev.com/28f1a1eacc3d88986bf5b92d1acdc17983891834/third_party/WebKit/Source/devtools/front_end/ui/SplitWidget.js

Comment 8 by l...@chromium.org, Jan 17 2018

Status: Fixed (was: Assigned)

Sign in to add a comment