Page MenuHomePhabricator

Update Settings to use full side-navigation
ClosedPublic

Authored by chad on Aug 4 2017, 5:19 PM.
Tags
None
Referenced Files
F18805973: D18342.id.diff
Sat, Oct 18, 5:11 PM
F18797651: D18342.diff
Fri, Oct 17, 5:17 AM
F18795149: D18342.id44096.diff
Fri, Oct 17, 1:39 AM
F18758513: D18342.id44095.diff
Sun, Oct 5, 10:37 PM
F18754848: D18342.id.diff
Sun, Oct 5, 1:45 AM
F18736828: D18342.diff
Wed, Oct 1, 7:56 AM
F18658240: D18342.diff
Sep 23 2025, 5:28 AM
F18651724: D18342.diff
Sep 21 2025, 5:11 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.