Skip to content

Show a hint when filters hide every row instead of a blank table - #53

Merged
laomou merged 1 commit into
mainfrom
feat/empty-result-message
Aug 6, 2026
Merged

Show a hint when filters hide every row instead of a blank table#53
laomou merged 1 commit into
mainfrom
feat/empty-result-message

Conversation

@laomou

@laomou laomou commented Aug 6, 2026

Copy link
Copy Markdown
Owner

Summary

When a log is loaded but the active filters match nothing, the table body rendered zero rows — a blank panel identical whether the cause was a typo'd Find term, a stale column filter (PID/TID/tag/level), or "Bookmarks only" with no bookmarks. There was no in-context explanation of why the table was empty.

Now it renders a centered "No rows match the active filters" hint (localized) in the remainder column, which always fills the width so the text isn't clipped. This is distinct from the empty-model state (no file / cleared), which keeps showing the keyboard-shortcut hints.

Test plan

  • all_rows_filtered_out_renders_no_matches_hint_without_panic — a non-matching Find hides every row and the hint renders without panic
  • All 104 tests pass
  • Manual: load a file, type a Find term that matches nothing → hint shows; clear it → rows return

When entries are loaded but the active filters match nothing, the table
body rendered zero rows — an empty panel identical whether caused by a
typo'd Find, a stale column filter, or "bookmarks only" with no bookmarks,
with no visible cause. Render a "No rows match the active filters" hint in
the remainder column (which always fills the width, so it isn't clipped).
Distinct from the empty-model view, which keeps showing the shortcut hints.
@laomou
laomou merged commit 1bdcbd5 into main Aug 6, 2026
3 checks passed
@laomou
laomou deleted the feat/empty-result-message branch August 6, 2026 09:27
laomou added a commit that referenced this pull request Aug 6, 2026
The "no rows match the active filters" hint (from #53) sat in one table
column a few rows down, which read as awkward. Instead, when entries are
loaded but the active filters hide every one of them, render the status-bar
"filtered 0" count itself in bold orange — compact, and it points right at
the count that explains the empty table. The table area stays clean and the
now-unused no_matches string is removed.
laomou added a commit that referenced this pull request Aug 6, 2026
The "no rows match the active filters" hint (from #53) sat in one table
column a few rows down, which read as awkward. Instead, when entries are
loaded but the active filters hide every one of them, render the status-bar
"filtered 0" count itself in bold orange — compact, and it points right at
the count that explains the empty table. The table area stays clean and the
now-unused no_matches string is removed.
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