Skip to content

fix(ui): corrects tab badge counts, toolbar layout, and ignore flash#42

Merged
wgordon17 merged 6 commits intogordon-code:mainfrom
wgordon17:fix/tab-badge-counts-toolbar-layout
Apr 2, 2026
Merged

fix(ui): corrects tab badge counts, toolbar layout, and ignore flash#42
wgordon17 merged 6 commits intogordon-code:mainfrom
wgordon17:fix/tab-badge-counts-toolbar-layout

Conversation

@wgordon17
Copy link
Copy Markdown
Member

Summary

  • Tab badges now exclude permanently hidden items (Dependency Dashboard, ignored items, PR-triggered runs) to match what each tab actually displays
  • Toolbar layout pins expand/collapse and ignore badge to top-right instead of wrapping to their own row on narrow viewports
  • Reorder highlight no longer flashes when ignoring/un-ignoring items

- adds globalFilter (org/repo) to tabCounts memo so badges match displayed items

- passes per-type ignored count to createReorderHighlight instead of total

- makes getIgnoredCount parameter required, removes optional chaining

- uses consistent block-form filter style across all tabCounts branches

- adds comprehensive reorderHighlight tests covering suppression and resumption

- adds tests for PR/run ignore badge decrement and showPrRuns toggle

- uses precise digit-extraction assertions for badge count verification
Covers cor-1 fix: verifies tabCounts filters by viewState.globalFilter.repo
Covers QA domain review findings: lockedChanged branch coverage and org prefix filter path
PR/Actions un-ignore round-trip, showPrRuns+ignore combo, timer clear
@wgordon17 wgordon17 merged commit 046b511 into gordon-code:main Apr 2, 2026
1 check passed
@wgordon17 wgordon17 deleted the fix/tab-badge-counts-toolbar-layout branch April 8, 2026 17:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant