Homec4science

Fix anchor-clicking scroll positions

Authored by epriestley <git@epriestley.com> on Jan 28 2015, 17:26.

Description

Fix anchor-clicking scroll positions

Summary:
Fixes T7069. When jumping to a comment anchor, we get the scroll positions wrong.

Partly this is fixing some calcaulations; partly, the "show older comments" and "scroll anchor" stuff were fighting over the scroll position. Since the anchor can take care of things on its own, just let it handle stuff.

Test Plan:

  • Clicked comment anchors.
  • Loaded pages with anchors in the URI.
  • Loaded pages with anchors hidden behind "show older comments".

In all cases, got the right scroll position.

Reviewers: btrahan, chad

Reviewed By: chad

Subscribers: epriestley

Maniphest Tasks: T7069

Differential Revision: https://secure.phabricator.com/D11540

Details

Committed
epriestley <git@epriestley.com>Jan 28 2015, 17:26
Pushed
aubortJan 31 2017, 17:16
Parents
rPH48f15fb96089: Update stock Homepage to look like Dashboards
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHeaa883cf37e3: Fix anchor-clicking scroll positions (authored by epriestley <git@epriestley.com>).Jan 28 2015, 17:26