fix(sidebar): prioritize actionable overflow activity - #7134
Draft
tellaho wants to merge 3 commits into
Draft
Conversation
Co-authored-by: Rizz <302abe414ca6e3134763d2539bfcf145aea2a63fe5f8455204ed602fd40cf381@buzz.block.builderlab.xyz> Co-authored-by: Carl <acda9e433d19dcd0e6b6840f7f4b98f3a56f1fab98049d444c087019e6d36560@buzz.block.builderlab.xyz> Signed-off-by: Taylor Ho <taylorkmho@gmail.com>
🔐 Codex Security Review
|
Co-authored-by: Rizz <302abe414ca6e3134763d2539bfcf145aea2a63fe5f8455204ed602fd40cf381@buzz.block.builderlab.xyz> Signed-off-by: Taylor Ho <taylorkmho@gmail.com>
Contributor
Author
|
Corrected the overflow presentation in
Regression evidence at |
Co-authored-by: Rizz <302abe414ca6e3134763d2539bfcf145aea2a63fe5f8455204ed602fd40cf381@buzz.block.builderlab.xyz> Signed-off-by: Taylor Ho <taylorkmho@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Category: fix
User Impact: Sidebar overflow controls stay quiet for ordinary activity and become prominent only when offscreen activity includes an unread direct message or mention.
Problem: The sidebar gave ordinary offscreen activity the same visual urgency as unread DMs and mentions, making the control louder than the work it represented. Its count also needed to remain honest when the prominent state was triggered by only part of the unread set.
Solution: Derive urgency from the existing high-priority unread projection while preserving the total unread/activity count and the newer unread-DM navigation behavior.
File changes
desktop/src/app/AppShell.tsx
Passes the existing high-priority unread channel set into the sidebar.
desktop/src/features/sidebar/lib/useOffscreenActivityChannelIds.test.mjs
Covers the message-only channel projection used to decide whether offscreen activity is actionable.
desktop/src/features/sidebar/lib/useSidebarActivityOverflow.ts
Determines per-direction urgency from offscreen message channels while retaining ordered channel IDs for DM navigation.
desktop/src/features/sidebar/ui/AppSidebar.tsx
Selects quiet or prominent overflow treatment per direction without disturbing unread-DM targeting.
desktop/src/features/sidebar/ui/AppSidebar.types.ts
Adds the high-priority unread channel set to the sidebar contract.
desktop/src/features/sidebar/ui/MoreUnreadButton.test.mjs
Covers the integrated unread-DM preview behavior.
desktop/src/features/sidebar/ui/MoreUnreadButton.tsx
Accepts explicit visual emphasis while preserving DM avatars, accessibility text, and target behavior.
Reproduction steps