Closed Bug 1515946 (scroll-anchoring-release) Opened 7 years ago Closed 7 years ago

Let scroll anchoring ride the trains for all platforms but Fennec

Categories

(Core :: Layout, defect, P2)

Product:
Core
Component:
Layout
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla66
Tracking Flags:
Tracking Status
firefox66 + fixed

People

(Reporter: rhunt, Assigned: rhunt)

References

(Depends on 1 open bug, Blocks 1 open bug)

Details

(Keywords: dev-doc-complete, site-compat, Whiteboard: [geckoview:fenix:p2])

Attachments

(1 file)

Bug 1515946 - Let scroll anchoring ride the trains on everything except Fennec. r?dholbert
47 bytes, text/x-phabricator-request
Details | Review
This bug is to track the work to let scroll anchoring ride the trains.
Priority: -- → P3

Scroll anchoring will be very important on mobile because screens are small so a page position jump will be more disruptive.

Whiteboard: [geckoview:fenix:p2]

Changing the priority to p2 as the bug is tracked by a release manager for the current nightly.
See How Do You Triage for more information

Priority: P3 → P2
Keywords: site-compat
Alias: scroll-anchoring-release
Blocks: scroll-anchoring
Depends on: 1519541
Depends on: 1519510
Depends on: 1519462
Depends on: 1519399
Blocks: 1519553
Blocks: 1518624
No longer blocks: 1518624
Depends on: 1518624
No longer blocks: 1519553
Depends on: 1519553
Depends on: 1518626
Depends on: 1518631
Depends on: 1518926
Depends on: 1518640
Depends on: 1518637
Depends on: 1518636
Depends on: 1518633

I haven't triaged all of these bugs to see whether they should actually block release. I'll start dropping these as we figure out their causes.

Depends on: 1519798
No longer depends on: 1518637
No longer depends on: 1518636
No longer depends on: 1518633
No longer depends on: 1518626
No longer depends on: 1518631
Depends on: 1520095
Depends on: 1520304
Depends on: 1520124
Depends on: 1520666
No longer depends on: 1519553
No longer depends on: 1518640
Pushed by rhunt@eqrion.net: https://hg.mozilla.org/integration/autoland/rev/f1694d9e640b Let scroll anchoring ride the trains on everything except Fennec. r=dholbert

Should we keep it enabled for Fennec Nightly?

Flags: needinfo?(rhunt)

https://hg.mozilla.org/mozilla-central/rev/f1694d9e640b

Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox66: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla66
Assignee: nobody → rhunt

I considered that, but didn't feel strongly that it needs to remain on. I also don't feel strongly that it needs to be turned off either, I guess.

If we can find a solution to the dynamic toolbar problem relatively quickly, then I think it makes sense to leave it on in nightly.

Flags: needinfo?(rhunt)
No longer depends on: 1521812
Summary: Let scroll anchoring ride the trains → Let scroll anchoring ride the trains for all platforms but Fennec
Depends on: 1522964
Depends on: 1523063
Depends on: 1526776

We did have this listed as enabled in Fx66 on both desktop and mobile, but then I saw this bug. I've updated the compat data and rel notes to correct this:

https://github.com/mdn/browser-compat-data/pull/3707
https://developer.mozilla.org/en-US/docs/Mozilla/Firefox/Releases/66#CSS

I hope that's OK, thanks!

Keywords: dev-doc-complete
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: