Fixes T11493. This code is a little bit weird/clever, simplify it so that we always cast the handles to an array early on.
Details
Details
Diff Detail
Diff Detail
- Repository
- rP Phabricator
- Branch
- hlist1
- Lint
Lint Passed - Unit
Tests Passed - Build Status
Buildable 13366 Build 17152: Run Core Tests Build 17151: arc lint + arc unit
Event Timeline
Comment Actions
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.
Comment Actions
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.