Page MenuHomePhabricator

Rewrite workboards to have way more bugs
ClosedPublic

Authored by epriestley on Feb 10 2016, 6:45 PM.
Tags
None
Referenced Files
F13092316: D15234.diff
Thu, Apr 25, 3:40 AM
Unknown Object (File)
Sat, Apr 20, 10:03 PM
Unknown Object (File)
Sun, Apr 14, 10:49 PM
Unknown Object (File)
Thu, Apr 11, 10:49 AM
Unknown Object (File)
Thu, Apr 11, 3:44 AM
Unknown Object (File)
Thu, Apr 11, 3:44 AM
Unknown Object (File)
Thu, Apr 11, 3:44 AM
Unknown Object (File)
Thu, Apr 11, 3:44 AM
Subscribers
None

Details

Summary

Ref T4900. Briefly:

  • Much more layout and rendering is now done in Javascript.
  • This should otherwise be identical to the behavior at HEAD, except that:
    • editing a task and removing the current board from it no longer removes the task; and
    • points still don't work.

However, this can now plausibly support realtime workboard updates and other complex state-based behaviors like points calculations in a future change.

Test Plan
  • Changed card covers.
  • Moved cards.
  • Sorted board by priority and natural.
  • Added new cards.
  • Edited cards in place.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Rewrite workboards to have way more bugs.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
  • Slightly smaller/cleaner diff.
chad edited edge metadata.
chad added inline comments.
src/applications/project/view/ProjectBoardTaskCard.php
114

I feel like we should move this into PHUIObjectItemView as a setType or something?

This revision is now accepted and ready to land.Feb 10 2016, 7:10 PM

It's only in one place now (sort of) so I'm not too worried about it.

This revision was automatically updated to reflect the committed changes.