Differential D17222

Attempt to make dropdown item actions more consistent
ClosedPublic
Actions

Authored by epriestley on Jan 18 2017, 7:22 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Dec 13, 12:45 PM

	Unknown Object (File)
	Fri, Dec 6, 6:21 PM

	Unknown Object (File)
	Fri, Dec 6, 2:45 PM

	Unknown Object (File)
	Thu, Nov 28, 1:35 AM

	Unknown Object (File)
	Wed, Nov 20, 8:28 PM

	Unknown Object (File)
	Wed, Nov 20, 10:31 AM

	Unknown Object (File)
	Wed, Nov 20, 10:31 AM

	Unknown Object (File)
	Wed, Nov 20, 10:08 AM

Subscribers

None

Details

Reviewers

Commits

rP45c3aaeb269e: 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.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Jan 18 2017, 7:22 PM

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

Harbormaster completed remote builds in B15312: Diff 41423.Jan 18 2017, 7:23 PM

Actually, something else already broke that.

C2a4am-UkAAlG7X.jpg (750×750 px, 55 KB)

After rebasing on D17223, I think everything (including the submenus in "Actions" on mobile on tasks) works correctly.

Harbormaster completed remote builds in B15314: Diff 41425.Jan 18 2017, 7:34 PM

chad accepted this revision.Jan 18 2017, 8:33 PM

This revision is now accepted and ready to land.Jan 18 2017, 8:33 PM

Closed by commit rP45c3aaeb269e: Attempt to make dropdown item actions more consistent (authored by epriestley, committed by epriestley). · Explain WhyJan 18 2017, 9:14 PM

This revision was automatically updated to reflect the committed changes.

epriestley mentioned this in rP90258ed491d5: Fix property shadowing on ActionListView after conversion to TagView.

Revision Contents
Changeset List

Path

Size

resources/

celerity/

66 lines

webroot/

rsrc/

js/

application/

projects/

behavior-project-boards.js

7 lines

phui/

behavior-phui-dropdown-menu.js

14 lines

phuix/

PHUIXActionView.js

2 lines

PHUIXDropdownMenu.js

28 lines

Diff 41432

resources/celerity/map.php

Loading...

webroot/rsrc/js/application/projects/behavior-project-boards.js

Loading...

webroot/rsrc/js/phui/behavior-phui-dropdown-menu.js

Loading...

webroot/rsrc/js/phuix/PHUIXActionView.js

Loading...

webroot/rsrc/js/phuix/PHUIXDropdownMenu.js

Loading...