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)
Wed, Jul 2, 1:02 AM
Unknown Object (File)
Sun, Jun 29, 2:20 PM
Unknown Object (File)
Jun 1 2025, 5:25 PM
Unknown Object (File)
May 31 2025, 6:55 PM
Unknown Object (File)
May 31 2025, 5:23 PM
Unknown Object (File)
May 31 2025, 12:13 AM
Unknown Object (File)
May 29 2025, 8:58 AM
Unknown Object (File)
May 6 2025, 1:37 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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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