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)
Thu, Apr 11, 10:37 AM
Unknown Object (File)
Thu, Apr 11, 1:40 AM
Unknown Object (File)
Mon, Apr 8, 2:04 AM
Unknown Object (File)
Fri, Apr 5, 8:56 AM
Unknown Object (File)
Fri, Mar 29, 6:25 AM
Unknown Object (File)
Mar 15 2024, 5:57 PM
Unknown Object (File)
Feb 3 2024, 9:41 PM
Unknown Object (File)
Jan 25 2024, 1:53 AM
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
Branch
grip10
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 22252
Build 30429: Run Core Tests
Build 30428: arc lint + arc unit

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.