HomePhabricator

Attempt to make dropdown item actions more consistent

Description

Attempt to make dropdown item actions more consistent

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.

Reviewers: chad

Reviewed By: chad

Differential Revision: https://secure.phabricator.com/D17222

Details

Provenance
epriestleyAuthored on Jan 18 2017, 7:17 PM
epriestleyPushed on Jan 18 2017, 9:14 PM
Reviewer
chad
Differential Revision
D17222: Attempt to make dropdown item actions more consistent
Parents
rP90258ed491d5: Fix property shadowing on ActionListView after conversion to TagView
Branches
Unknown
Tags
Unknown
Build Status
Buildable 15321
Build 20169: Run Core Tests