Page MenuHomePhabricator

Provide better UI feedback about cards that can't be dragged or edited
ClosedPublic

Authored by epriestley on Mar 11 2019, 6:01 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Mar 15, 5:57 PM
Unknown Object (File)
Feb 3 2024, 9:41 PM
Unknown Object (File)
Jan 25 2024, 1:53 AM
Unknown Object (File)
Jan 1 2024, 4:26 PM
Unknown Object (File)
Dec 28 2023, 4:36 PM
Unknown Object (File)
Dec 26 2023, 11:44 AM
Unknown Object (File)
Dec 24 2023, 4:46 PM
Unknown Object (File)
Dec 23 2023, 11:36 PM
Subscribers
None

Details

Summary

Depends on D20273. Fixes T10722. Currently, we don't make it very clear when a card can't be edited. Long ago, some code made a weak attempt to do this (by hiding the "grip" on the card), but later UI changes hid the "grip" unconditionally so that mooted things.

Instead:

  • Replace the edit pencil with a red lock.
  • Provide cursor hints for grabbable / not grabbable.
  • Don't let users pick up cards they can't edit.
Test Plan

On a workboard with a mixture of editable and not-editable cards, hovered over the different cards and was able to figure out which ones I could drag or not drag pretty easily. Picked up cards I could pick up, wasn't able to drag cards I can't edit.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

amckinley added inline comments.
webroot/rsrc/css/phui/workboards/phui-workcard.css
91–93

This should also clearly make a loud bonging noise if the user attempts to drag a card anyway.

This revision is now accepted and ready to land.Mar 12 2019, 8:07 PM
webroot/rsrc/css/phui/workboards/phui-workcard.css
91–93

GREAT idea

HUGE ui/ux improvement

This revision was automatically updated to reflect the committed changes.