Homec4science

Allow draggable elements to be dragged to the first list position

Authored by epriestley <git@epriestley.com> on Sep 25 2013, 22:27.

Description

Allow draggable elements to be dragged to the first list position

Summary:
Currently, draggable lists (in Config and ApplicationSearch, for example) don't let you drag an item into the first position.

This is because the behavior is correct in Maniphest: the first position is above an initial header, like "High Prioirty", and shouldn't be targetable.

Permit the behavior in general; forbid it in Maniphest.

Test Plan:

  • Dragged elements into the first position in ApplicationSearch.
  • Failed to drag elements into the first position in Maniphest.

Reviewers: garoevans, btrahan

Reviewed By: garoevans

CC: aran

Differential Revision: https://secure.phabricator.com/D7128

Details

Committed
epriestley <git@epriestley.com>Sep 25 2013, 22:27
Pushed
aubortJan 31 2017, 17:16
Parents
rPH3361eba13907: Padding on Maniphest status
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH21bd596d7103: Allow draggable elements to be dragged to the first list position (authored by epriestley <git@epriestley.com>).Sep 25 2013, 22:27