Page MenuHomePhabricator
Differential D6845

[web] Style navigation state info in the top bar
ClosedPublic
Actions

Authored by inka on Feb 23 2023, 5:14 AM.
Tags
None
Referenced Files
F3537739: D6845.id23684.diff
Wed, Dec 25, 10:12 PM
F3537738: D6845.id23667.diff
Wed, Dec 25, 10:12 PM
F3537737: D6845.id23532.diff
Wed, Dec 25, 10:12 PM
F3537736: D6845.id22987.diff
Wed, Dec 25, 10:12 PM
F3537654: D6845.id.diff
Wed, Dec 25, 10:10 PM
F3537551: D6845.diff
Wed, Dec 25, 9:59 PM
Unknown Object (File)
Thu, Nov 28, 9:02 AM
Unknown Object (File)
Thu, Nov 28, 6:21 AM
View All Files
Subscribers
ashoat
atul
tomek

Details

Reviewers
kamil
michal
bartek
Commits
rCOMM966d977bbe17: [web] Style navigation state info in the top bar
Summary

issue: https://linear.app/comm/issue/ENG-2740/add-navigation-state-info-to-the-top-bar
Styling the nav state info according to the designs: https://www.figma.com/file/a1nkbWgbgjRlrOY9LVurTz/Comm-%2F-Desktop-app?node-id=7780%3A136895&t=zg5f0X86eUyYnqI7-4

image.png (246×2 px, 37 KB)

image.png (252×2 px, 35 KB)

image.png (244×2 px, 36 KB)

Test Plan

Run web app, check that the navigation state info in the top bar displays correctly

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

inka created this revision.Feb 23 2023, 5:14 AM
Herald added subscribers: atul, tomek, ashoat. · View Herald TranscriptFeb 23 2023, 5:14 AM
inka edited the summary of this revision. (Show Details)Feb 23 2023, 5:15 AM
inka edited the test plan for this revision. (Show Details)
inka attached a referenced file: F398087: Screen Recording 2023-02-23 at 13.34.36.mov. (Show Details)
inka added a parent revision: D6842: [web] Add navigation state info to the topbar.
inka added a child revision: D6848: [web] Fix filter panel showing over the topbar.Feb 23 2023, 5:28 AM
Harbormaster completed remote builds in B16813: Diff 22987.Feb 23 2023, 5:30 AM
inka requested review of this revision.Feb 23 2023, 5:30 AM
inka planned changes to this revision.Feb 27 2023, 12:07 AM

The idea to have the overflow: scroll was not accepted by the design team, so I'll have to update that once we decide what to do instead.

inka updated this revision to Diff 23532.Mar 8 2023, 3:26 AM

Change overflow: scroll to hidden. It has been decided that we want to "bias" towards the last item, hence the "flex-shrink: 0.5" for the last item

inka edited the summary of this revision. (Show Details)Mar 8 2023, 3:32 AM
Harbormaster completed remote builds in B17182: Diff 23532.Mar 8 2023, 3:42 AM
kamil accepted this revision.Mar 13 2023, 2:37 AM
This revision is now accepted and ready to land.Mar 13 2023, 2:37 AM
inka mentioned this in D6745: [web] Split ThreadTopBar into two components.Mar 13 2023, 4:05 AM
inka updated this revision to Diff 23667.Mar 13 2023, 4:10 AM

Remove unnnecessary div

Harbormaster completed remote builds in B17278: Diff 23667.Mar 13 2023, 4:25 AM
Closed by commit rCOMM966d977bbe17: [web] Style navigation state info in the top bar (authored by • InkaSokolowska). · Explain WhyMar 13 2023, 6:35 AM
This revision was automatically updated to reflect the committed changes.
• InkaSokolowska added a commit: rCOMM966d977bbe17: [web] Style navigation state info in the top bar.
inka mentioned this in D6393: Fixed incorret display for long thread names on web..Mar 14 2023, 7:37 AM

Revision Contents
Changeset List

Diff 23667

View Options

web/navigation-panels/chat-thread-ancestors.css

Loading...
View Options

web/navigation-panels/chat-thread-ancestors.react.js

Loading...
View Options

web/navigation-panels/nav-state-info-bar.css

Loading...
View Options

web/navigation-panels/nav-state-info-bar.react.js

Loading...
View Options

web/theme.css

Loading...