Page MenuHomePhabricator
Differential D10186

When dragging stuff around, compensate for changes in scroll position
ClosedPublic
Actions

Authored by epriestley on Aug 7 2014, 11:41 PM.
Tags
None
Referenced Files
F18846799: D10186.diff
Wed, Oct 29, 9:10 PM
F18818761: D10186.id24558.diff
Wed, Oct 22, 2:12 AM
F18818760: D10186.id24503.diff
Wed, Oct 22, 2:12 AM
F18718474: D10186.id.diff
Sep 29 2025, 4:01 PM
F18703005: D10186.id.diff
Sep 28 2025, 2:32 AM
F18699229: D10186.id24558.diff
Sep 27 2025, 12:33 PM
F18697238: D10186.diff
Sep 27 2025, 9:18 AM
F18503949: D10186.id.diff
Sep 4 2025, 11:30 PM
View All Files
Subscribers
epriestley

Details

Reviewers
btrahan
Maniphest Tasks
T5240: Dragging items (cards, panels) could be better in Workboards
Commits
Restricted Diffusion Commit
rP0e98e33b3342: When dragging stuff around, compensate for changes in scroll position
Summary

Ref T5240. This fixes dragging stuff and using the mousewheel or keyboard to scroll the window during the drag.

(It does not fix "dragging near the edge of the container should scroll it" yet.)

Test Plan

Dragged stuff around on task list and workboards in Safari, Firefox and Chrome. Used mousewheel and shift + mousewheel to scroll the document and containers during drag. Object remained under the cursor.

Diff Detail

Repository
rP Phabricator
Branch
draggy
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 2098
Build 2102: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 24503.Aug 7 2014, 11:41 PM
epriestley retitled this revision from to When dragging stuff around, compensate for changes in scroll position.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
epriestley added a task: T5240: Dragging items (cards, panels) could be better in Workboards.
Herald added a subscriber: epriestley. · View Herald TranscriptAug 7 2014, 11:41 PM
Harbormaster completed remote builds in B2098: Diff 24503.Aug 7 2014, 11:43 PM
btrahan accepted this revision.Aug 8 2014, 10:35 PM
btrahan edited edge metadata.
btrahan added inline comments.
webroot/rsrc/externals/javelin/lib/Vector.js
312

Nice. This scroll stuff is a pain in the butt and this will help a ton.

This revision is now accepted and ready to land.Aug 8 2014, 10:35 PM
epriestley closed this revision.Aug 9 2014, 3:49 PM
epriestley updated this revision to Diff 24558.

Closed by commit rP0e98e33b3342 (authored by @epriestley).

Revision Contents
Changeset List

PathSize
resources/
celerity/
34 lines
webroot/
rsrc/
externals/
javelin/
lib/
29 lines
js/
core/
10 lines

Diff 24503

View Options

resources/celerity/map.php

Loading...
View Options

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

Loading...
View Options

webroot/rsrc/js/core/DraggableList.js

Loading...
Phabricator · Built by Phacility