Removes many tables and uses PropertyLists and ObjectItemList when possible. Adds cleaner CSS, makes mobile editing more possible.
Details
Details
- Reviewers
epriestley btrahan - Maniphest Tasks
- Restricted Maniphest Task
- Commits
- Restricted Diffusion Commit
rPc453e98c4093: Moderize Herald UI
Test new UI on desktop and mobile. Verify all functionality still exists.
Diff Detail
Diff Detail
- Repository
- rP Phabricator
- Branch
- herald-yo-momma
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 67 Build 67: [Placeholder Plan] Wait for 30 Seconds
Event Timeline
Comment Actions
Nice!
One minor thing: instead of "View Transcript" as a link, I think the transcript's main link should probably go to the transcript, and then we should either put like D1234 and link it in the attributes or just omit that entirely. I mess this up fairly often on the current UI (i.e., you 99% of the time want to go to the transcript from this UI, and it's confusing that the biggest link doesn't take you there). We can easily tweak this later, though.