Page scrolled to bottom on load (or pageshow)
Categories
(Core :: DOM: Selection, defect, P2)
Tracking
()
People
(Reporter: jscher2000, Assigned: masayuki)
References
()
Details
(Keywords: parity-chrome, parity-edge, regression)
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
Updated•7 years ago
|
Assignee | ||
Comment 1•7 years ago
|
||
Assignee | ||
Updated•7 years ago
|
Updated•7 years ago
|
Comment 3•7 years ago
|
||
Updated•6 years ago
|
Comment 4•6 years ago
|
||
Updated•6 years ago
|
Updated•6 years ago
|
Updated•6 years ago
|
Assignee | ||
Updated•6 years ago
|
Assignee | ||
Updated•6 years ago
|
Assignee | ||
Comment 5•6 years ago
|
||
Assignee | ||
Comment 6•6 years ago
|
||
Assignee | ||
Comment 7•6 years ago
|
||
Assignee | ||
Comment 8•6 years ago
|
||
When Selection changed into an editing host,
Selection::NotifySelectionListeners() moves focus to the editing host.
In this case, we've scrolled to the focused element because it's our consistent
and traditional behavior. However, Chrome does not behave so. Therefore,
we should not scroll in this case for compatibility with Chrome.
(Although, I've not succeeded to run the WPTs on Chrome with
./mach wpt --product chrome
due to (probably) under testing/web-platform/mozilla/tests
,
as far as I've tested, Chrome won't scroll to focused editing host at least
when it gets focus because of Selection API.)
Comment 10•6 years ago
|
||
bugherder |
Comment 11•6 years ago
|
||
This fix has been verified on Nightly v67.0a1 from 2019-02-21 on Windows 10, Mac OS 10.13.6 and Ubuntu 16.04 LTS.
Comment 12•6 years ago
|
||
Is this something to be considered for uplift or should it just ride the trains?
Assignee | ||
Comment 14•6 years ago
|
||
(In reply to Ryan VanderMeulen [:RyanVM] from comment #12)
Is this something to be considered for uplift or should it just ride the trains?
The patch should be safe, but this is not so urgent bug since we get only this report and bug 1490126. So, not many web apps are broken by this incompatible issue. Perhaps, for risk management, it's better to make it just ride the train.
Updated•6 years ago
|
Description
•