Page MenuHomePhabricator
Differential D11523

Fix inline reticles and dialogs for new scrollbars
ClosedPublic
Actions

Authored by epriestley on Jan 27 2015, 2:59 PM.
Tags
None
Referenced Files
F18815724: D11523.id27710.diff
Tue, Oct 21, 3:24 AM
F18788996: D11523.id27711.diff
Wed, Oct 15, 7:29 AM
F18764362: D11523.id.diff
Tue, Oct 7, 5:39 AM
F18761884: D11523.diff
Mon, Oct 6, 4:39 PM
F18475940: D11523.diff
Sep 3 2025, 12:03 AM
F17923162: D11523.id.diff
Jul 30 2025, 7:48 AM
F17912474: D11523.diff
Jul 29 2025, 4:18 PM
Unknown Object (File)
Jun 21 2025, 3:58 PM
View All Files
Subscribers
epriestley

Details

Reviewers
btrahan
chad
Maniphest Tasks
T7049: Dialogs always move page back to top
T7054: Scrolling page after highlighting inlines scrolls the reticle, too
Commits
Restricted Diffusion Commit
rP41dcd7420b5b: Fix inline reticles and dialogs for new scrollbars
Summary

Fixes T7054. Fixes T7049.

  • Stop scrolling Differential reticles when the page scrolls.
  • Make dialogs aware of multi-panel UI.
Test Plan
  • Dialogs pop up in the right place.
  • Inline + scroll now longer leaves the inline in a fixed position.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 27710.Jan 27 2015, 2:59 PM
epriestley retitled this revision from to Fix inline reticles and dialogs for new scrollbars.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: chad, btrahan.
epriestley added tasks: T7054: Scrolling page after highlighting inlines scrolls the reticle, too, T7049: Dialogs always move page back to top.
Herald added a subscriber: epriestley. · View Herald TranscriptJan 27 2015, 2:59 PM
chad accepted this revision.Jan 27 2015, 3:03 PM
chad edited edge metadata.
This revision is now accepted and ready to land.Jan 27 2015, 3:03 PM
Closed by commit rP41dcd7420b5b: Fix inline reticles and dialogs for new scrollbars (authored by epriestley, committed by epriestley). · Explain WhyJan 27 2015, 3:11 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
celerity/
50 lines
webroot/
rsrc/
externals/
javelin/
lib/
8 lines
js/
application/
differential/
14 lines

Diff 27711

View Options

resources/celerity/map.php

Loading...
View Options

webroot/rsrc/externals/javelin/lib/Workflow.js

Loading...
View Options

webroot/rsrc/js/application/differential/behavior-edit-inline-comments.js

Loading...
Phabricator · Built by Phacility