Page MenuHomePhabricator

Simplify how tag lists manage their handles
ClosedPublic

Authored by epriestley on Aug 19 2016, 5:45 PM.
Tags
None
Referenced Files
F15437012: D16422.diff
Tue, Mar 25, 5:01 PM
F15421003: D16422.id39491.diff
Fri, Mar 21, 8:03 PM
F15397306: D16422.id39492.diff
Sun, Mar 16, 7:29 PM
F15390384: D16422.id39491.diff
Sat, Mar 15, 6:10 AM
Unknown Object (File)
Feb 18 2025, 11:36 AM
Unknown Object (File)
Feb 9 2025, 2:11 AM
Unknown Object (File)
Feb 9 2025, 2:11 AM
Unknown Object (File)
Feb 9 2025, 2:11 AM
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.