Page MenuHomePhabricator
Differential D6842

[web] Add navigation state info to the topbar
ClosedPublic
Actions

Authored by inka on Thu, Feb 23, 1:05 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Mar 17, 3:59 PM
Unknown Object (File)
Wed, Mar 8, 9:11 PM
Unknown Object (File)
Tue, Mar 7, 1:59 PM
Unknown Object (File)
Sun, Mar 5, 9:34 AM
Unknown Object (File)
Fri, Feb 24, 6:06 AM
Unknown Object (File)
Fri, Feb 24, 6:05 AM
Unknown Object (File)
Thu, Feb 23, 12:26 PM
F397540: Screen Recording 2023-02-21 at 15.33.35.mov
Thu, Feb 23, 1:05 AM
Subscribers
ashoat
atul
tomek

Details

Reviewers
kamil
michal
bartek
Commits
rCOMMf50f8f28a5ba: [web] Add navigation state info to the topbar
Summary

issue: https://linear.app/comm/issue/ENG-2740/add-navigation-state-info-to-the-top-bar
Add NavStateInfoBar to the topbar.

Test Plan

Run web app, checked that the navigation state info shows up in the topbar, and the animations work correclty.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

inka created this revision.Thu, Feb 23, 1:05 AM
Herald added subscribers: atul, tomek, ashoat. · View Herald TranscriptThu, Feb 23, 1:05 AM
inka edited the summary of this revision. (Show Details)Thu, Feb 23, 1:05 AM
inka attached a referenced file: F397540: Screen Recording 2023-02-21 at 15.33.35.mov. (Show Details)
inka added a parent revision: D6841: [web] Add PossiblyEmptyNavStateInfoBar to handle collapse animation in the topbar when no community is selected.
Harbormaster completed remote builds in B16795: Diff 22964.Thu, Feb 23, 1:19 AM
inka requested review of this revision.Thu, Feb 23, 1:19 AM
inka added a child revision: D6845: [web] Style navigation state info in the top bar.Thu, Feb 23, 5:16 AM
inka mentioned this in D6848: [web] Fix filter panel showing over the topbar.Thu, Feb 23, 5:28 AM
bartek accepted this revision.Fri, Feb 24, 5:45 AM
This revision is now accepted and ready to land.Fri, Feb 24, 5:45 AM
Closed by commit rCOMMf50f8f28a5ba: [web] Add navigation state info to the topbar (authored by • InkaSokolowska). · Explain WhyMon, Mar 13, 6:35 AM
This revision was automatically updated to reflect the committed changes.
• InkaSokolowska added a commit: rCOMMf50f8f28a5ba: [web] Add navigation state info to the topbar.
• InkaSokolowska mentioned this in rCOMM4bd6b6c4efa6: [web] Fix filter panel showing over the topbar.

Revision Contents
Changeset List

PathSize
web/
navigation-panels/
19 lines

Diff 23683

View Options

web/navigation-panels/topbar.react.js

Loading...