Skip to content

feat: AI review UI redesign - #4050

Merged
bdshadow merged 5 commits into
bdshadow/ai-review-rulesfrom
zuzanaodstrcilova/ai-review-ux-v2
Nov 13, 2026
Merged

bdshadow merged 5 commits into
bdshadow/ai-review-rulesfrom
zuzanaodstrcilova/ai-review-ux-v2

Conversation

@ZuzanaOdstrcilova

Copy link
Copy Markdown
Collaborator

UI redesign of the AI review frontend on top of the AI review PR. Visual changes only: every action, permission and data flow stays as it was.

What changes

Shared pill: StatusBadge (component/common)

  • Tones success / warning / error / teal / neutral, tinted or outlined, 22px, 13px text.
  • Colours come from the Tolgee tokens and work in light and dark mode.
  • The AI score badge in the translation cell and the issue severity badge (Critical / Major / Minor) use it. It can serve TM matches later.

AI review card (popover)

  • "AI review score" with the number on the right.
  • One outcome line: Auto-marked as reviewed / Stays translated (needs N+) / Stays reviewed + Translation protection / Score only / Reviewed by… or Set back to translated by… (overridden).
  • The rule and time sit on one line; prompt and model are in the rule tooltip.
  • Issues sit under "Issue" / "Issues · N points off".
  • A left stripe follows the translation state.
  • Failures use the standard error Alert with an outlined Retry.
  • A running re-review shows a grey ShimmerText.
  • The title and issues are shared components (AiReviewScoreTitle, AiReviewIssues).

Rule rows

  • One filter is shown by name; two or more, or one that does not fit, as "N filters" with the list and prompt in a tooltip.
  • Counts are icon + number: reviewed, waiting, failed.
  • The waiting count is still the per-rule "review waiting" button (same action and data-cy).
  • "Retry failed" is grey.
  • On narrow screens the counts wrap under the scope.

Review prompt preview

  • Uses the same score title and issues as the card.
  • The preview box has a 20px star and 16px on the right.
  • Prompt type chips use TaskTypeChip.

Tests

  • Lint and typecheck are clean, and the vitest unit tests pass.
  • E2E: two tests changed with the UI:
    • badge.cy.ts: prompt and model are in the rule tooltip, so the test hovers the rule first.
    • ruleDialog.cy.ts: three filters show as "3 filters", so the test checks the tooltip.
    • dataCyType.d.ts is regenerated and only adds selectors.
  • The E2E suite was not run locally (no docker compose); please rely on CI.

Notes

  • New translation keys only have default values in code; en.json is untouched.

🤖 Generated with Claude Code

ZuzanaOdstrcilova and others added 4 commits November 12, 2026 20:35
Add StatusBadge, a small pill for statuses and results (AI score, issue
severity, later TM matches) with tones and colours from the Tolgee tokens
for light and dark mode. The AI score badge in the translation cell and the
issue severity badge use it; the cell keeps a 12px gap to the other icons.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The card shows "AI review score" with the number, one short outcome line
(auto-marked, stays translated, stays reviewed with Translation protection,
score only, overridden by a person), the rule and time, and the issues under
"Issue"/"Issues" with the points off. A left stripe follows the translation
state. Failures use Tolgee's error Alert with an outlined Retry, a running
re-review a grey shimmer. Title and issues are shared components.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
One filter is shown by name, two or more (or one that does not fit) as
"N filters" with the list and prompt in a tooltip. Counts are icons with
numbers: reviewed, waiting (also runs them) and failed, then a grey
"Retry failed". On narrow screens the counts wrap under the scope.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The review prompt preview uses the same score title and issues as the AI
review card; the preview box has a 20px star and more room on the right.
Prompt type chips in the prompt list use the task type chip.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Nov 12, 2026 •

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

Run configuration

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

@bdshadow
bdshadow merged commit 720951a into bdshadow/ai-review-rules Nov 13, 2026
51 checks passed
@bdshadow
bdshadow deleted the zuzanaodstrcilova/ai-review-ux-v2 branch November 13, 2026 09:03
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.

2 participants