HomePhabricator

Fix anchor-clicking scroll positions

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

Provenance
epriestleyAuthored on
epriestleyPushed on Jan 28 2015, 4:26 PM
Reviewer
chad
Differential Revision
D11540: Fix anchor-clicking scroll positions
Parents
rP48f15fb96089: Update stock Homepage to look like Dashboards
Branches
Unknown
Tags
Unknown
Tasks
T7069: Scrolling offsets of anchor jumps is incorrect