Page MenuHomePhabricator

Update Settings to use full side-navigation
ClosedPublic

Authored by chad on Aug 4 2017, 5:19 PM.
Tags
None
Referenced Files
F15468356: D18342.id44095.diff
Thu, Apr 3, 11:44 PM
F15463909: D18342.id44096.diff
Wed, Apr 2, 4:24 AM
F15462918: D18342.id.diff
Tue, Apr 1, 6:56 PM
F15462496: D18342.diff
Tue, Apr 1, 3:28 PM
F15458773: D18342.diff
Mon, Mar 31, 6:53 AM
F15414265: D18342.diff
Wed, Mar 19, 11:48 PM
F15382145: D18342.diff
Mar 14 2025, 10:57 AM
F15375147: D18342.diff
Mar 12 2025, 9:52 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
Branch
settings-new-ui-aug-2017 (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 17910
Build 24054: Run Core Tests
Build 24053: arc lint + arc unit

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.