Page MenuHomePhabricator

Update Settings to use full side-navigation
ClosedPublic

Authored by chad on Aug 4 2017, 5:19 PM.
Tags
None
Referenced Files
F19303963: D18342.id44096.diff
Dec 24 2025, 6:32 AM
F19191875: D18342.diff
Dec 16 2025, 10:03 AM
F19014694: D18342.diff
Nov 22 2025, 9:09 PM
F18998598: D18342.diff
Nov 20 2025, 7:09 PM
F18992438: D18342.diff
Nov 19 2025, 3:02 AM
F18805973: D18342.id.diff
Oct 18 2025, 5:11 PM
F18797651: D18342.diff
Oct 17 2025, 5:17 AM
F18795149: D18342.id44096.diff
Oct 17 2025, 1:39 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.