Page MenuHomePhabricator
Differential D6337

[web] Move topbar related code out of web/sidebar
AbandonedPublic
Actions

Authored by inka on Jan 22 2023, 11:22 AM.
Tags
None
Referenced Files
F1669533: D6337.diff
Sat, Apr 27, 11:02 AM
Unknown Object (File)
Wed, Apr 24, 2:37 AM
Unknown Object (File)
Tue, Apr 23, 10:19 AM
Unknown Object (File)
Wed, Apr 3, 4:14 PM
Unknown Object (File)
Wed, Apr 3, 4:12 PM
Unknown Object (File)
Feb 20 2024, 5:42 AM
Unknown Object (File)
Feb 18 2024, 1:49 AM
Unknown Object (File)
Feb 17 2024, 11:31 PM
View All 18 Files
Subscribers
ashoat
atul
tomek

Details

Reviewers
bartek
• przemek
michal
Summary

Since D6335 the AppSwitcher, SettingsSwitcher and NavigationPanel are no longer in a sidebar, but in a topbar. So the code related to them had to be moved outside of web/sidebar. Moreover since D6336 the LeftLayoutAside doesn't exist
anymore, so the left-layout-aside.css file had been renamed to topbar.css.

Test Plan

Run the web app, check that the topbar displayes correctly.

Diff Detail

Repository
rCOMM Comm
Branch
inka/community_drawer_web
Lint
No Lint Coverage
Unit
No Test Coverage

Revision Contents
Changeset List

Diff 21172

View Options

web/app.react.js

Loading...
View Options

web/sidebar/app-switcher.react.js

Loading...
View Options

web/sidebar/left-layout-aside.css

Loading...
View Options

web/sidebar/navigation-panel.react.js

Loading...
View Options

web/sidebar/settings-switcher.react.js

Loading...
View Options

web/topbar/app-switcher.react.js

Loading...
View Options

web/topbar/navigation-panel.react.js

Loading...
View Options

web/topbar/settings-switcher.react.js

Loading...
View Options

web/topbar/topbar.css

Loading...