Implement more correct white-icon menu behavior
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F19797013: D10967.diff
	Fri, Feb 27, 6:06 PM

	F19785980: D10967.id26337.diff
	Wed, Feb 25, 4:34 AM

	F19785979: D10967.id.diff
	Wed, Feb 25, 4:34 AM

	F19686661: D10967.id26336.diff
	Mon, Feb 9, 1:17 PM

	F19531911: D10967.id.diff
	Jan 20 2026, 7:47 AM

	F19528204: D10967.id.diff
	Jan 19 2026, 7:45 AM

	F19275173: D10967.id26337.diff
	Dec 23 2025, 12:30 AM

	F19181263: D10967.id26336.diff
	Dec 16 2025, 2:42 AM

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