fix(web): stop the filter count appearing twice on the transactions toolbar - #1097
Merged
Conversation
Contributor
Greptile SummaryRemoves the redundant active-filter count badge from the transactions toolbar while retaining the descriptive “N active” row and clear action.
Confidence Score: 5/5The code change appears safe to merge, though repository policy still requires the unsigned commits to be replaced with verified commits. No blocking code failure remains in the reviewed change. Important Files Changed
Reviews (8): Last reviewed commit: "Merge branch 'main' into fix/filter-coun..." | Re-trigger Greptile |
tobySolutions
force-pushed
the
fix/filter-count-badge-contrast
branch
from
August 5, 2026 12:42
17253c0 to
5f601dc
Compare
GuiBibeau
approved these changes
Aug 5, 2026
This was referenced Aug 6, 2026
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.
The transactions toolbar rendered the active filter count twice on the same condition: a badge on the Filters button, and an "N active" line directly beneath it. The badge was the weaker of the two — a bare number crowded against the button edge, beside a line that says what the number means and offers to clear it.
It was also unreadable in dark mode: it set
text-whiteagainstbg-primary, which resolves near-white there, so the count sat white on white.Removes the badge and keeps the line.
Note: #962 also touches this file, in a different region.