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

Branch

scroll9

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 4157
Build 4170: [Placeholder Plan] Wait for 30 Seconds

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

14 lines

webroot/

rsrc/

js/

application/

diffusion/

behavior-jump-to.js

3 lines

Diff 27820

View Options

resources/celerity/map.php