Page MenuHomePhabricator

Implement more correct white-icon menu behavior
ClosedPublic

Authored by epriestley on Dec 11 2014, 12:20 AM.
Tags
None
Referenced Files
F18808750: D10967.diff
Sun, Oct 19, 8:22 AM
F18761436: D10967.id26336.diff
Mon, Oct 6, 2:20 PM
F18748606: D10967.id.diff
Fri, Oct 3, 11:39 PM
F18741834: D10967.diff
Thu, Oct 2, 4:17 PM
F18626064: D10967.id.diff
Sep 16 2025, 12:16 AM
F18599185: D10967.diff
Sep 13 2025, 6:53 AM
F18596246: D10967.id26337.diff
Sep 13 2025, 12:08 AM
F18588220: D10967.id26337.diff
Sep 11 2025, 11:36 PM
Subscribers

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 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.
chad edited edge metadata.

haha

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

sorry you had to write more JS

This revision was automatically updated to reflect the committed changes.