Paths

Table of Contentst

Fix scrolling behavior in Diffusion
ClosedPublic
Actions

Authored by epriestley on Jan 29 2015, 6:19 PM.

Details

Reviewers

btrahan
chad

Maniphest Tasks

T7084: navigate-to-on-load fail with(?) fancy scrollbars

Commits

Restricted Diffusion Commit
rP7ed4bb4f176e: Fix scrolling behavior in Diffusion

Summary

Fixes T7084. This doesn't use the same anchor logic as other applications.

Test Plan

$245 lines now jump to line 245 on page load.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 27820.Jan 29 2015, 6:19 PM

epriestley retitled this revision from to Fix scrolling behavior in Diffusion.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added reviewers: chad, btrahan.

epriestley added a task: T7084: navigate-to-on-load fail with(?) fancy scrollbars.

Herald added a subscriber: epriestley. · View Herald TranscriptJan 29 2015, 6:19 PM

chad accepted this revision.Jan 29 2015, 6:20 PM

chad edited edge metadata.

This revision is now accepted and ready to land.Jan 29 2015, 6:20 PM

Closed by commit rP7ed4bb4f176e: Fix scrolling behavior in Diffusion (authored by epriestley, committed by epriestley). · Explain WhyJan 29 2015, 6:20 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

resources/

celerity/

map.php

24 lines

webroot/

rsrc/

js/

application/

diffusion/

behavior-jump-to.js

3 lines

Diff 27821

View Options

resources/celerity/map.php