Page MenuHomePhabricator

Add ActionIcon to PHUIListItemView, use in Dashboards
ClosedPublic

Authored by chad on Mar 16 2017, 6:21 PM.
Tags
None
Referenced Files
F14035413: D17505.id42098.diff
Sun, Nov 10, 5:49 AM
F14035333: D17505.id42096.diff
Sun, Nov 10, 5:31 AM
F14035069: D17505.diff
Sun, Nov 10, 3:58 AM
F14032170: D17505.diff
Sat, Nov 9, 1:12 PM
F14022386: D17505.id.diff
Wed, Nov 6, 4:58 PM
F13966893: D17505.id.diff
Wed, Oct 16, 10:20 AM
F13963934: D17505.diff
Tue, Oct 15, 6:42 PM
Unknown Object (File)
Oct 1 2024, 9:57 PM
Subscribers

Details

Summary

Extends PHUIListItemView to take an icon, link as an "Action Item" that displays on the right side of the menu link. Does not display on Favorites. This allows for adding edit, external, or other links (documentation?) to any menu item. Right now the secondary link is only visible when the item is selected. This feels right, but if we offer it in other ways, users may always want it visible. We could look at making it onhover.

Test Plan

Add a bunch of random global and personal dashboards to my menu. Add a menu to Favorites, see no link. Test mobile, link works.

pasted_file (718×664 px, 73 KB)

Diff Detail

Repository
rP Phabricator
Branch
dashboard-edit (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 16005
Build 21220: Run Core Tests
Build 21219: arc lint + arc unit

Event Timeline

  • test for dashboards named "MMMMMMMMMMMMMMMMMMMMMMMMM" by users named hector
This revision is now accepted and ready to land.Mar 16 2017, 6:30 PM
This revision was automatically updated to reflect the committed changes.