Page MenuHomePhabricator

Rough initial cut of profile new profile menu
ClosedPublic

Authored by epriestley on Jan 14 2016, 1:04 AM.
Tags
None
Referenced Files
F14047098: D15016.id.diff
Thu, Nov 14, 1:58 AM
F13973797: D15016.id36268.diff
Oct 18 2024, 2:45 AM
F13958405: D15016.id36301.diff
Oct 14 2024, 2:37 PM
Unknown Object (File)
Oct 3 2024, 6:33 PM
Unknown Object (File)
Sep 21 2024, 7:00 AM
Unknown Object (File)
Sep 20 2024, 10:13 AM
Unknown Object (File)
Sep 11 2024, 11:57 AM
Unknown Object (File)
Sep 4 2024, 1:17 AM
Subscribers
None

Details

Summary

Ref T10054. I haven't done any of the big-picture layout stuff yet, but this should get look-and-feel somewhere in the ballpark of reasonablness, I think.

Major missing stuff:

  • No "collapse" state or action yet.
  • Menu is not full-height (requires changes to the rendering pipeline).
Test Plan

Screen Shot 2016-01-13 at 4.59.39 PM.png (1×1 px, 206 KB)

Diff Detail

Repository
rP Phabricator
Branch
pmenu2
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 10102
Build 12243: Run Core Tests
Build 12242: arc lint + arc unit

Event Timeline

epriestley retitled this revision from to Rough initial cut of profile new profile menu.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.

I'll hold this until full-height is also built/approved since the partial-height is super weird. I probably got colors/layouts not-quite-right, feel free to counterdiff on anything that looks a bit off.

chad edited edge metadata.
This revision is now accepted and ready to land.Jan 14 2016, 9:06 PM
This revision was automatically updated to reflect the committed changes.