Page MenuHomePhabricator

[ROUGH!] Fix some drag-and-drop issues with Workboards
AbandonedPublic

Authored by epriestley on Dec 23 2015, 9:32 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Jan 25, 8:41 PM
Unknown Object (File)
Fri, Jan 24, 10:47 AM
Unknown Object (File)
Fri, Jan 24, 10:47 AM
Unknown Object (File)
Fri, Jan 24, 10:47 AM
Unknown Object (File)
Tue, Jan 7, 12:47 PM
Unknown Object (File)
Tue, Jan 7, 12:47 PM
Unknown Object (File)
Tue, Jan 7, 12:46 PM
Unknown Object (File)
Tue, Jan 7, 8:29 AM
Subscribers
None

Details

Summary

Ref T5240. This needs some more work but maybe kick the tires on it and see how it feels?

Basically just JS atrocities:

  • Move the header nodes around in desktop vs mobile to give us fixed headers.
  • Resize stuff all the time in JS.

Things that are kind of bad:

  • Vertical scrollbar is odd and not easily fixable. Maybe just, uh, hide it completely?
  • I ungood'ed some of the design.
  • You can't drag cards out of the board area. This doesn't work at HEAD either but needs to work before we can do the drag-to-a-tab UI from M1450.
  • Icon nav is bleeding into the board now because of reasons.
Test Plan

board.png (767×1 px, 80 KB)

Diff Detail

Repository
rP Phabricator
Branch
proj6
Lint
Lint Warnings
SeverityLocationCodeMessage
Warningwebroot/rsrc/js/application/projects/behavior-phantom-headers.js:1JAVELIN5`javelinsymbols` Not In Path
Unit
Tests Passed
Build Status
Buildable 9755
Build 11719: Run Core Tests
Build 11718: arc lint + arc unit

Event Timeline

epriestley retitled this revision from to [ROUGH!] Fix some drag-and-drop issues with Workboards.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley edited edge metadata.

This one should apply to master now, I think.

Yeah, this is rough. What feedback are you looking for?

Mostly like "does this seem 100% terrible", I guess?

It feels not great, but mostly just CSS stuff that needs cleaned and polished. I also think maybe we can remove the icon nav and use a menu icon like in PHUIDocumentPro.