When a user drags a card over a column, highlight the column border
ClosedPublic
Actions

Authored by epriestley on Mar 2 2019, 3:49 PM.

Details

Reviewers

amckinley

Maniphest Tasks

T10334: On workboards, distinguish between "drag to column" and "drag to specific place in column"

Commits

rPbe1e3b2cc0a8: When a user drags a card over a column, highlight the column border

Summary

Ref T10334. Partly, this just improves visual feedback for all drag operations. After D20242, we can have cases where you (for example) drag a low-priority node to a very tall column on a priority-ordered workboard. In this case, the actual dashed-border-drop-target may not be on screen.

We might make the column scroll or put some kind of hint in the UI in this case, but an easy starting point is just to make the "yes, you're targeting this column" state a bit more clear.

Test Plan

Dragged tasks between columns, saw the border higlight on the target columns. This is very tricky to take a screenshot of.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Mar 2 2019, 3:49 PM

Harbormaster completed remote builds in B22173: Diff 48312.Mar 2 2019, 3:51 PM

epriestley requested review of this revision.Mar 2 2019, 3:51 PM

This is very tricky to take a screenshot of.

What, phone's broken?

webroot/rsrc/css/phui/workboards/phui-workpanel.css
142	Doesn't the new "ghost" task box have a dashed border?

This revision is now accepted and ready to land.Mar 7 2019, 6:43 PM

There are four separate UI feedback indicators right now:

The card in the old position becomes partly transparent until you "let go" of it.
A transparent copy of the card is put under the cursor. I think the code usually calls this the "ghost", although I may not be very consistent about it.
A dashed purple "drop target" (which probably also gets called a "ghost" sometimes) is rendered into the target column if releasing your mouse would move the card somewhere else.
If releasing your mouse would move the card to a column (possibly the same column), the whole column which the card would move to (or "stay in" but move to a different position in) now gets a relatively subtle solid border.

The first three are pre-existing, this just adds (4). I think it'll make the most sense to just try dragging something once this hits master. It also doesn't help that phui-box-grey is not actually grey much of the time.

Closed by commit rPbe1e3b2cc0a8: When a user drags a card over a column, highlight the column border (authored by epriestley). · Explain WhyMar 9 2019, 6:30 PM

This revision was automatically updated to reflect the committed changes.