Console doesn't scroll to bottom
Reported by
fabian.z...@gmail.com,
Mar 12 2016
|
|||||||||
Issue descriptionUserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_3) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/49.0.2623.87 Safari/537.36 Steps to reproduce the problem: 1. Debug an JS-App with the console open 2. 3. What is the expected behavior? Console scrolls down to bottom when new messages arrive What went wrong? My console stops scrolling after some messages… Did this work before? N/A Chrome version: 49.0.2623.87 Channel: stable OS Version: OS X 10.11.3 Flash Version: Shockwave Flash 21.0 r0
,
Mar 12 2016
Looks like this is the intended behavior from fixing issue 580381 . Assigning to lushnikov to confirm.
,
Mar 14 2016
So there's no autoscroll-to-bottom anymore?
,
Mar 14 2016
Yea, auto-scroll is gone. However, I noticed in testing some stuff the other day that the console jitters as if it doesn't want to go to the end in some cases. Trying to get a reproduction case going for it...
,
Mar 14 2016
I think the sticking issue comes from logs that wrap. Steps to repro: 1. Shrink your dev console so the following string wraps, and run in your dev console: for (var i = 0; i < 1000; ++i) console.log(i + 'Lorem ipsum dolor sit amet, consectetur adipiscing elit. Integer nunc lacus, tincidunt ac ullamcorper sed, sagittis eget lectus. Nam vel tempor lacus. Quisque sit amet sodales turpis, eget sagittis dui. Cras accumsan massa sed risus sodales fermentum. Praesent non vestibulum mauris, nec lobortis.'); 2. Observe the console follows for the first few logs, then loses the bottom and stops following On the other hand, running `for (var i = 0; i < 1000; ++i) console.log(i);` does work
,
Mar 14 2016
#3, #4: There is autoscroll-to-bottom. It should work. #5: correct, the issue comes from the wrapped console messages. Thank you!
,
Apr 6 2016
Is there any estimate on when a fix will be available? This is really annoying for developers!
,
Apr 6 2016
Also happens when outputting HTML elements: https://output.jsbin.com/naquxaqoca
,
Apr 6 2016
,
Apr 7 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/1d413c3ebced31d7182b31ff6efb458c5a12216e commit 1d413c3ebced31d7182b31ff6efb458c5a12216e Author: lushnikov <lushnikov@chromium.org> Date: Thu Apr 07 21:28:44 2016 DevTools: [Console] fix console stick-to-bottom behavior in case of multiline messages The patch fixes the stick-to-bottom behavior of console in case of multiline messages. The patch does two things: 1. ViewportControl starts to cache provider's itemCount in invalidate call. This ensures that the regular scrolling of viewport is not affected by provider's item changes unless an invalidate method was called. 2. Stick to bottom only in case of non-user-generated gestures. This is done via transitioning to the event.isTrusted flag. BUG= 594339 R=dgozman, pfeldman Review URL: https://codereview.chromium.org/1863313002 Cr-Commit-Position: refs/heads/master@{#385872} [modify] https://crrev.com/1d413c3ebced31d7182b31ff6efb458c5a12216e/third_party/WebKit/LayoutTests/inspector/console/console-viewport-selection.html [modify] https://crrev.com/1d413c3ebced31d7182b31ff6efb458c5a12216e/third_party/WebKit/LayoutTests/inspector/console/console-viewport-stick-to-bottom-expected.txt [modify] https://crrev.com/1d413c3ebced31d7182b31ff6efb458c5a12216e/third_party/WebKit/LayoutTests/inspector/console/console-viewport-stick-to-bottom.html [modify] https://crrev.com/1d413c3ebced31d7182b31ff6efb458c5a12216e/third_party/WebKit/Source/devtools/front_end/ui/ViewportControl.js
,
Apr 8 2016
It would be nice to have this in M50.
,
Apr 8 2016
Has this been baked in canary and verified?
,
Apr 8 2016
[Automated comment] Less than 2 weeks to go before stable on M50, manual review required.
,
Apr 8 2016
Verified working on Mac chrome canary
,
Apr 8 2016
Merge approved for M50 (branch 2661). Pls go ahead merge.
,
Apr 9 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/7c8faba3e1586366ae97bba310f0aa6df9b14281 commit 7c8faba3e1586366ae97bba310f0aa6df9b14281 Author: Andrey Lushnikov <lushnikov@chromium.org> Date: Sat Apr 09 00:09:12 2016 DevTools: [Console] fix console stick-to-bottom behavior in case of multiline messages The patch fixes the stick-to-bottom behavior of console in case of multiline messages. The patch does two things: 1. ViewportControl starts to cache provider's itemCount in invalidate call. This ensures that the regular scrolling of viewport is not affected by provider's item changes unless an invalidate method was called. 2. Stick to bottom only in case of non-user-generated gestures. This is done via transitioning to the event.isTrusted flag. BUG= 594339 R=dgozman, pfeldman Review URL: https://codereview.chromium.org/1863313002 Cr-Commit-Position: refs/heads/master@{#385872} (cherry picked from commit 1d413c3ebced31d7182b31ff6efb458c5a12216e) Review URL: https://codereview.chromium.org/1875853002 . Cr-Commit-Position: refs/branch-heads/2661@{#538} Cr-Branched-From: ef6f6ae5e4c96622286b563658d5cd62a6cf1197-refs/heads/master@{#378081} [modify] https://crrev.com/7c8faba3e1586366ae97bba310f0aa6df9b14281/third_party/WebKit/LayoutTests/inspector/console/console-viewport-selection.html [modify] https://crrev.com/7c8faba3e1586366ae97bba310f0aa6df9b14281/third_party/WebKit/LayoutTests/inspector/console/console-viewport-stick-to-bottom-expected.txt [modify] https://crrev.com/7c8faba3e1586366ae97bba310f0aa6df9b14281/third_party/WebKit/LayoutTests/inspector/console/console-viewport-stick-to-bottom.html [modify] https://crrev.com/7c8faba3e1586366ae97bba310f0aa6df9b14281/third_party/WebKit/Source/devtools/front_end/ui/ViewportControl.js
,
Apr 9 2016
,
Apr 13 2016
Tested the issue on Windows 7, Mac 10.10.5, Ubuntu 14.04 using 50.0.2661.75.Observed that console scrolls down to bottom when new messages arrive. Please find attached screencast. Marking it as TE-Verified.
,
Apr 13 2016
Tested with Version 52.0.2707.0 canary (64-bit) on Mac OS X 10.10.5. Doesn't fix scroll issue as can be observed in https://output.jsbin.com/qefeso
,
Apr 13 2016
@jhu: it is fixed for long wrapping messages; embedded nodes contribute to a different scenario, which is not currently supported. I've filed a separate issue for that: https://crbug.com/603294 |
|||||||||
►
Sign in to add a comment |
|||||||||
Comment 1 by fabian.z...@gmail.com
, Mar 12 2016145 KB
145 KB View Download