Page MenuHomePhabricator

Update Settings to use full side-navigation
ClosedPublic

Authored by chad on Aug 4 2017, 5:19 PM.
Tags
None
Referenced Files
F14043044: D18342.diff
Tue, Nov 12, 7:25 AM
F14025488: D18342.diff
Thu, Nov 7, 6:25 PM
F14005525: D18342.id44095.diff
Sun, Oct 27, 2:46 PM
F14002970: D18342.id44095.diff
Sat, Oct 26, 12:26 AM
F13975453: D18342.id44096.diff
Fri, Oct 18, 10:11 AM
F13961481: D18342.id44096.diff
Oct 15 2024, 5:16 AM
F13958455: D18342.id44095.diff
Oct 14 2024, 2:51 PM
Unknown Object (File)
Oct 13 2024, 3:44 AM
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.