New issue
Advanced search Search tips

Issue 799784 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner: ----
Closed: Jan 11
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 2
Type: Bug



Sign in to add a comment

Fr units in grid template causing errors while appending elements

Reported by aarek...@gmail.com, Jan 7 2018

Issue description

UserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/65.0.3314.0 Safari/537.36

Steps to reproduce the problem:
1. Open fr.html
2. Scroll down, see how new elements append
3. Open calc.html
4. Scroll down, compare with fr.html experience

What is the expected behavior?
Every time you scroll close to the bottom, 10 new elements are appended to the container, just like in case shown in calc.html

What went wrong?
Using fr units, more than 10 elements are appended every time you scroll close to the bottom

Did this work before? No 

Does this work in other browsers? Yes

Chrome version: 65.0.3314.0  Channel: canary
OS Version: 10.0
Flash Version: 

Both cases in form of jsfiddle:
fr.html: https://jsfiddle.net/vr790bqe/1/
calc.html: https://jsfiddle.net/vr790bqe/2/
 
fr.html
1.8 KB View Download
calc.html
1.8 KB View Download
Labels: Needs-Triage-M65

Comment 2 by e...@chromium.org, Jan 7 2018

Labels: Needs-Feedback
Unable to reproduce, both examples add ten items each time the scroll position triggers the condition. Given that the number of items added is hard-coded "appendChildren(10);
" I don't see how it could ever add any other number of items.

Comment 3 by aarek...@gmail.com, Jan 7 2018

It was more colloquial term, it's coded to add exactly 10 elements at once. The problem is with scroll position.

Let me repharse - the expected effect is to have infinite scroll in the container element. Using fr units it is indeed achieved, but instead of adding another 10 elements under current view and keeping scroll position above them, it adds 10 elements, but doesn't keep `scrollTop` position, hence elements keep getting added. Please compare both examples (calc.html works like expected result, it's also how it works in other browsers).

Sorry for not mentioning scrollTop behaviour in original issue, figured it out only recently.

Attaching the demo with `scrollTop` logging, as well as video demonstration.
fr_logging.html
1.9 KB View Download
calc_logging.html
1.9 KB View Download
demo_fr.mp4
186 KB View Download
demo_calc.mp4
317 KB View Download
Project Member

Comment 4 by sheriffbot@chromium.org, Jan 7 2018

Cc: e...@chromium.org
Labels: -Needs-Feedback
Thank you for providing more feedback. Adding requester "eae@chromium.org" to the cc list and removing "Needs-Feedback" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 5 by e...@chromium.org, Jan 7 2018

Cc: szager@chromium.org
Status: Available (was: Unconfirmed)
Thanks!
Project Member

Comment 6 by sheriffbot@chromium.org, Jan 8

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
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
Status: WontFix (was: Untriaged)
No longer reproduces.

Sign in to add a comment