Skip to content

ui(results): groupings empty state uses the UI kit empty state (#1162) - #1229

Merged
ZhuchkaTriplesix merged 1 commit into
devfrom
feat/1162-groupings-empty-state
Oct 9, 2026
Merged

ZhuchkaTriplesix merged 1 commit into
devfrom
feat/1162-groupings-empty-state

Conversation

@ZhuchkaTriplesix

Copy link
Copy Markdown
Member

Refs #1162

What was wrong

The groupings view showed No data available for grouping. as a bare muted line, the last raw empty state in the results (#1162 asks for QueryaEmptyState in the results).

Changes

  • The empty groupings view is a QueryaEmptyState: "No data to group", with a hint to run a query that returns rows.
  • The existing groupings test checks the new title.

Not in this PR

  • Copying the chart as an image in the export menu (needs a platform clipboard call for images).
  • The guard test covers the four files named in the issue, not all of lib/features/.

Not verified locally

Tests were not run locally, per the project rule.

@github-actions github-actions Bot added enhancement New feature or request frontend Theme parser epic label: frontend ui User interface components and widgets data-grid Interactive data grid, cell editor, filtering, groupings P2 Medium priority / Parity & Refactoring labels Oct 9, 2026
@ZhuchkaTriplesix
ZhuchkaTriplesix merged commit cde7ce8 into dev Oct 9, 2026
14 checks passed
@github-actions

github-actions Bot commented Oct 9, 2026

Copy link
Copy Markdown

Grid scroll benchmark

metric base PR change
p50 17.41 ms 15.82 ms -9.1%
p90 24.50 ms 28.12 ms +14.8% ⚠️
p99 43.92 ms 48.79 ms +11.1% ⚠️
stutters 285.00 286.00 +0.4%

Informational only (threshold 5%). Shared CI runners are noisy; re-run before trusting a single result.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

data-grid Interactive data grid, cell editor, filtering, groupings enhancement New feature or request frontend Theme parser epic label: frontend P2 Medium priority / Parity & Refactoring ui User interface components and widgets

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant