Page MenuHomePhabricator

Update Settings to use full side-navigation
ClosedPublic

Authored by chad on Aug 4 2017, 5:19 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, May 3, 5:58 AM
Unknown Object (File)
Thu, Apr 25, 12:59 AM
Unknown Object (File)
Fri, Apr 19, 6:41 PM
Unknown Object (File)
Mon, Apr 15, 5:57 AM
Unknown Object (File)
Mon, Apr 15, 1:37 AM
Unknown Object (File)
Apr 10 2024, 3:44 PM
Unknown Object (File)
Apr 8 2024, 3:04 PM
Unknown Object (File)
Apr 8 2024, 3:03 PM
Subscribers

Details

Summary

Moves Settings to use a normal side navigation vs. a two column side navigation. It also updates Edit Engine to do the same, but I don't think there are other callsites. Added a consistent header for better clarification if you were editng your settings, global settings, or a bot's settings.

Test Plan

Test each page on a personal account, create global settings, test each page there, create a bot account, and test each page on the bot account. Anything else?

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Aug 4 2017, 5:20 PM
This revision was automatically updated to reflect the committed changes.

Is this "normal side navigation" used anywhere else? It looks a little strange:

image.png (443×432 px, 24 KB)

I expect to use that style in config and settings, places that are more 'under the hood' and need better navigation.