Page MenuHomePhabricator

Use standard tag elements instead of "maniphest project tag" to show tags in list views
ClosedPublic

Authored by epriestley on Jun 26 2014, 2:52 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 11, 11:52 PM
Unknown Object (File)
Mar 20 2024, 7:52 AM
Unknown Object (File)
Feb 16 2024, 3:13 PM
Unknown Object (File)
Feb 16 2024, 3:13 PM
Unknown Object (File)
Feb 16 2024, 3:13 PM
Unknown Object (File)
Feb 9 2024, 3:24 PM
Unknown Object (File)
Feb 3 2024, 9:12 PM
Unknown Object (File)
Feb 2 2024, 3:47 AM
Subscribers

Details

Summary

Standardizes tag rendering in Maniphest and Maniphest/Diffusion list views.

(This might need some size/spacing tweaks, I tried to make it look reasonable.)

Test Plan

See screenshots.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Use standard tag elements instead of "maniphest project tag" to show tags in list views.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.

My test data is pretty junk, but normal-ish view:

Screen_Shot_2014-06-26_at_7.47.52_AM.png (1×1 px, 181 KB)

With a column (I probably want to make this "Project > Column" in the tag eventually?):

Screen_Shot_2014-06-26_at_7.55.38_AM.png (350×1 px, 64 KB)

With too many projects:

Screen_Shot_2014-06-26_at_7.57.12_AM.png (1×1 px, 190 KB)

In Diffusion / restricted style:

Screen_Shot_2014-06-26_at_7.57.43_AM.png (120×1 px, 19 KB)

  • Fix a minor Celerity issue.

Looks OK. Also thought about the Typeahead, don't think it's worth even testing. Mostly because Typeaheads are interfaces for quickly selecting objects you already know, I don't think the color adds anything new.

chad edited edge metadata.
This revision is now accepted and ready to land.Jun 26 2014, 3:15 PM
epriestley updated this revision to Diff 23398.

Closed by commit rP41adc1b4c3db (authored by @epriestley).