Page MenuHomePhabricator

Simplify how tag lists manage their handles
ClosedPublic

Authored by epriestley on Aug 19 2016, 5:45 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Apr 22, 10:42 PM
Unknown Object (File)
Sun, Mar 31, 2:13 PM
Unknown Object (File)
Feb 12 2024, 8:59 AM
Unknown Object (File)
Jan 29 2024, 5:38 PM
Unknown Object (File)
Jan 28 2024, 1:07 PM
Unknown Object (File)
Jan 19 2024, 12:26 AM
Unknown Object (File)
Jan 7 2024, 5:11 AM
Unknown Object (File)
Dec 21 2023, 11:59 PM
Subscribers
None

Details

Summary

Fixes T11493. This code is a little bit weird/clever, simplify it so that we always cast the handles to an array early on.

Test Plan

Screen Shot 2016-08-19 at 10.42.44 AM.png (377×785 px, 55 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Simplify how tag lists manage their handles.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.

Should we make this "show all tags" instead? In case they want to be clicked on?

I think retaining the tooltip is probably good regardless. We could additionally let you click the (...) tag to pop up a dialog of the other tags, I guess, but since no one has ever asked for that it doesn't seem too valuable to me offhand.

chad edited edge metadata.
This revision is now accepted and ready to land.Aug 19 2016, 6:05 PM

Well, I don't know how useful the tooltip really is, actually. It seems sort of nice but it's possible that no one ever uses it.

I'd guess the individual tags on stuff with 5+ tags probably isn't all that useful most of the time, maybe? Not really sure.

This revision was automatically updated to reflect the committed changes.