Page MenuHomePhabricator
Differential D18805

Make bulk editor working set editable and more homogenous
ClosedPublic
Actions

Authored by epriestley on Nov 30 2017, 1:39 PM.
Tags
None
Referenced Files
F14053869: D18805.diff
Fri, Nov 15, 8:06 PM
F14044673: D18805.id.diff
Tue, Nov 12, 7:22 PM
F14044573: D18805.diff
Tue, Nov 12, 6:03 PM
F14042685: D18805.diff
Tue, Nov 12, 5:20 AM
F14027847: D18805.diff
Fri, Nov 8, 9:57 AM
F14015006: D18805.id45139.diff
Sun, Nov 3, 12:08 PM
F13989273: D18805.diff
Mon, Oct 21, 7:26 PM
F13984512: D18805.id45118.diff
Sun, Oct 20, 1:56 PM
View All Files
Subscribers
None

Details

Reviewers
amckinley
Maniphest Tasks
T10005: Implement an ApplicationEditor-based Bulk Editor
T11286: make it possible to remove tasks from a batch edit tasks workflow
T13025: Turn bulk editor into a generic piece of EditEngine infrastructure with Custom Field support
Commits
rPad659627b3e9: Make bulk editor working set editable and more homogenous
Summary

Ref T13025. See PHI50. Fixes T11286. Ref T10005. Begin modernizing the bulk editor.

For T10005 ("move the bulk editor to modern infrastructure"), rewrite the rendering of the editable set so that it is application-agnostic and can work with any kind of object.

For T11286 ("let users de-select items in the working set"), make the working set editable.

Test Plan

Screen Shot 2017-11-30 at 5.34.28 AM.png (1×1 px, 140 KB)

  • Deselected some objects, applied an edit, saw the edit apply to only selected objects.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Nov 30 2017, 1:39 PM
epriestley requested review of this revision.Nov 30 2017, 3:28 PM
epriestley added a child revision: D18806: Rebuild the bulk editor on SearchEngine.
amckinley accepted this revision.Nov 30 2017, 10:59 PM

New interface looks great!

webroot/rsrc/js/phui/behavior-phui-selectable-list.js
24

"selection"

This revision is now accepted and ready to land.Nov 30 2017, 10:59 PM
epriestley updated this revision to Diff 45139.Nov 30 2017, 11:18 PM
  • Spelling fix.
Closed by commit rPad659627b3e9: Make bulk editor working set editable and more homogenous (authored by epriestley). · Explain WhyJan 19 2018, 8:39 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
celerity/
17 lines
src/
applications/
maniphest/
controller/
77 lines
view/
phui/
43 lines
webroot/
rsrc/
css/
phui/
object-item/
19 lines
4 lines
js/
phui/
44 lines

Diff 45277

View Options

resources/celerity/map.php

Loading...
View Options

src/applications/maniphest/controller/ManiphestBatchEditController.php

Loading...
View Options

src/view/phui/PHUIObjectItemView.php

Loading...
View Options

webroot/rsrc/css/phui/object-item/phui-oi-list-view.css

Loading...
View Options

webroot/rsrc/css/phui/phui-box.css

Loading...
View Options

webroot/rsrc/js/phui/behavior-phui-selectable-list.js

Loading...
Phabricator · Built by Phacility