Page MenuHomePhabricator
Differential D6845

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

Authored by inka on Thu, Feb 23, 5:14 AM.
Tags
None
Referenced Files
F417011: image.png
Wed, Mar 8, 3:32 AM
F417010: image.png
Wed, Mar 8, 3:32 AM
F417009: image.png
Wed, Mar 8, 3:32 AM
Unknown Object (File)
Sat, Mar 4, 9:18 PM
Unknown Object (File)
Sat, Mar 4, 2:28 AM
Unknown Object (File)
Tue, Feb 28, 7:25 AM
Unknown Object (File)
Fri, Feb 24, 12:26 PM
Unknown Object (File)
Fri, Feb 24, 12:25 PM
View All 11 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
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

inka created this revision.Thu, Feb 23, 5:14 AM
Herald added subscribers: atul, tomek, ashoat. · View Herald TranscriptThu, Feb 23, 5:14 AM
inka edited the summary of this revision. (Show Details)Thu, Feb 23, 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.Thu, Feb 23, 5:28 AM
Harbormaster completed remote builds in B16813: Diff 22987.Thu, Feb 23, 5:30 AM
inka requested review of this revision.Thu, Feb 23, 5:30 AM
inka planned changes to this revision.Mon, Feb 27, 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.Wed, Mar 8, 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)Wed, Mar 8, 3:32 AM
Harbormaster completed remote builds in B17182: Diff 23532.Wed, Mar 8, 3:42 AM
kamil accepted this revision.Mon, Mar 13, 2:37 AM
This revision is now accepted and ready to land.Mon, Mar 13, 2:37 AM
inka mentioned this in D6745: [web] Split ThreadTopBar into two components.Mon, Mar 13, 4:05 AM
inka updated this revision to Diff 23667.Mon, Mar 13, 4:10 AM

Remove unnnecessary div

Harbormaster completed remote builds in B17278: Diff 23667.Mon, Mar 13, 4:25 AM
Closed by commit rCOMM966d977bbe17: [web] Style navigation state info in the top bar (authored by • InkaSokolowska). · Explain WhyMon, Mar 13, 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..Tue, Mar 14, 7:37 AM

Revision Contents
Changeset List

Diff 23684

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...