Implement more correct white-icon menu behavior
ClosedPublic
Actions

Authored by epriestley on Dec 11 2014, 12:20 AM.

Tags

None

Referenced Files

	F18914874: D10967.id26337.diff
	Nov 9 2025, 7:18 AM

	F18914871: D10967.id.diff
	Nov 9 2025, 7:18 AM

	F18906895: D10967.id.diff
	Nov 8 2025, 8:56 PM

	F18896539: D10967.diff
	Nov 7 2025, 4:10 PM

	F18862631: D10967.id26336.diff
	Nov 2 2025, 8:04 PM

	F18843753: D10967.id.diff
	Oct 29 2025, 12:40 AM

	F18808750: D10967.diff
	Oct 19 2025, 8:22 AM

	F18761436: D10967.id26336.diff
	Oct 6 2025, 2:20 PM

View All Files

Subscribers

epriestley

Details

Reviewers

chad

Commits

Restricted Diffusion Commit
rP3297bc2ed914: Implement more correct white-icon menu behavior

Summary

I didn't get this quite right.

Test Plan

Clicked to open, saw white, then closed by:
- Clicking document outside menu;
- clicking menu icon again;
- clicking a different menu icon.
In all three cases, got correct close + un-white behavior.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 26336.Dec 11 2014, 12:20 AM

epriestley retitled this revision from to Implement more correct white-icon menu behavior.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a reviewer: chad.

Herald added a subscriber: epriestley. · View Herald TranscriptDec 11 2014, 12:20 AM

haha

This revision is now accepted and ready to land.Dec 11 2014, 12:26 AM

sorry you had to write more JS

Closed by commit rP3297bc2ed914: Implement more correct white-icon menu behavior (authored by epriestley, committed by epriestley). · Explain WhyDec 11 2014, 12:26 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

resources/

celerity/

map.php

26 lines

webroot/

rsrc/

js/

application/

aphlict/

behavior-aphlict-dropdown.js

34 lines

Diff 26337

View Options

resources/celerity/map.php