Skip to content

fix(web): match highest accuracy card to neighboring blocks - #122

Merged
dibenkobit merged 2 commits into
mainfrom
fix/neutral-accuracy-card
Sep 9, 2026
Merged

dibenkobit merged 2 commits into
mainfrom
fix/neutral-accuracy-card

Conversation

@dibenkobit

@dibenkobit dibenkobit commented Sep 9, 2026 •

Copy link
Copy Markdown
Member

What changed

Use the same background and border as the neighboring blocks for the highest-accuracy card (black in dark mode, white in light mode). Keep the small green label accent.

Why

The large green fill looked muddy in dark mode and dominated the card.

Verification

  • Passed pnpm format:check, pnpm lint, pnpm --filter @llang-gap/web typecheck, and pnpm --filter @llang-gap/web build.
  • Checked EN language detail in dark and light themes at desktop and mobile widths; captured matching before/after screenshots. After screenshots use the production build.
  • Cosmetic CSS change; no new unit tests or documentation needed.

UI changes

Dark theme

EN before

EN after

Light theme

EN before

EN after

Checklist

  • This PR addresses one concern, and I reviewed the diff.
  • I explained the problem, the change, and the verification results.
  • I updated affected documentation and regression coverage, where applicable.
  • I attached UI evidence and checked the required language(s), where applicable.

View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

@vercel

vercel Bot commented Sep 9, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
langgap-web Ready Ready Preview Sep 9, 2026 8:57am UTC

Request Review

@dibenkobit dibenkobit changed the title fix(web): neutralize highest accuracy card background fix(web): match highest accuracy card to neighboring blocks Sep 9, 2026
@dibenkobit
dibenkobit merged commit e7894ca into main Sep 9, 2026
6 checks passed

This branch was successfully deployed

1 active deployment
Preview — 63aa0ad3 Deployed Sep 9, 2026 by vercel[bot]
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