Page MenuHomePhabricator

[web] Move code around to match the new code logic
ClosedPublic

Authored by inka on Jan 24 2023, 2:17 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Mar 12, 11:17 PM
Unknown Object (File)
Wed, Feb 28, 9:52 AM
Unknown Object (File)
Feb 24 2024, 9:10 AM
Unknown Object (File)
Feb 18 2024, 3:54 AM
Unknown Object (File)
Feb 18 2024, 3:10 AM
Unknown Object (File)
Feb 18 2024, 2:37 AM
Unknown Object (File)
Feb 18 2024, 1:19 AM
Unknown Object (File)
Feb 18 2024, 12:57 AM
Subscribers

Details

Summary

issue: https://linear.app/comm/issue/ENG-2739/move-navigationpanel-from-the-sidebar-to-a-top-bar
Now that the SettingSwitcher is no longer in the sidebar, it's code had to be moved outside of web/sidebar. The styles in left-layout-aside.css were only used for SettingsSwticher (and horizontal NavigationPanel that is used only in SettingsSwitcher so the file could be renamed and also moved. NavigationPanel is used for SettingSwitcher and AppSwitcher, so only for navigation panels. Thus all of this code could be put into web/navigation-panels as well as the Topbar, that is a navigation panel (contains AppSwitcher and will contain the navigation state information).

Test Plan

Run web app, check that the top bar in Inbox, and the SettingSwitcher in Settings display correctly. Run flow check in web/.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

inka requested review of this revision.Jan 24 2023, 2:31 AM
This revision is now accepted and ready to land.Feb 9 2023, 5:43 AM