Page MenuHomePhabricator

Attempt to make dropdown item actions more consistent
ClosedPublic

Authored by epriestley on Jan 18 2017, 7:22 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Jan 23, 11:26 PM
Unknown Object (File)
Tue, Jan 21, 3:20 PM
Unknown Object (File)
Tue, Jan 21, 12:28 PM
Unknown Object (File)
Dec 29 2024, 11:04 PM
Unknown Object (File)
Dec 27 2024, 10:33 AM
Unknown Object (File)
Dec 27 2024, 10:33 AM
Unknown Object (File)
Dec 27 2024, 10:21 AM
Unknown Object (File)
Dec 13 2024, 12:45 PM
Subscribers
None

Details

Summary

See D17210. Currently, this handler needs to be installed on each menu that doesn't build with the default behavior.

Rather than copy-pasting it to the user menu, try to make it a default behavior. This adds a new rule: don't close the menu if the item is a dynamic item built in JS with PHUIXActionView.

This allows dynamic items to control the menu themselves, while giving static items the desired default behavior.

Test Plan
  • Opened menus on: dashboards, user menu, timeline comments. Clicked stuff. Menus went away.
  • Other menus still seemed to work right: Diffusion, Favorites, mobile menu.

Diff Detail

Repository
rP Phabricator
Branch
actionlist2
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 15314
Build 20158: Run Core Tests
Build 20157: arc lint + arc unit

Event Timeline

No such luck: this breaks mobile "Action" menus on objects.

Actually, something else already broke that.

  • After rebasing on D17223, I think everything (including the submenus in "Actions" on mobile on tasks) works correctly.
This revision is now accepted and ready to land.Jan 18 2017, 8:33 PM