(Away) Reporter
	Description • 6 years ago

STR:
1. Set the old (left-aligned) theme
2. Open bug 1235982
3. Scroll so that the 85th comment is at the top of the screen (I am deliberately not linking to it, in case that interferes with the repro)
4. Click the link that refers to the 81st comment

Expected: Scrolls to 81st comment
Actual: Scrolls to roughly 73rd comment

	Emma Humphries ☕️🎸🧞‍♀️✨ (she/they) [:emceeaich] (Pacific Time) use needinfo
	Comment 1 • 6 years ago

What's our timeline to turn off the old (non-bug-modal) view? 

If we're getting rid of this soon, I'd wontfix this one.

If we're keeping it around, then this bug needs a priority.

	Kohei Yoshino [:kohei] Assignee
	Comment 3 • 6 years ago

The non-modal view should be removed sometime soon for further UX work, but I see the current script, namely the `offsetTop` property, is somewhat unreliable, so it could be broken in the modal view as well when CSS is modified. Will fix this anyway.

https://github.com/mozilla-bteam/bmo/blob/master/js/global.js#L221