Skip to content

fix: UI polish details and add audit skill#136

Merged
overtrue merged 1 commit into
mainfrom
overtrue/ui-audit-skill
Jun 15, 2026
Merged

fix: UI polish details and add audit skill#136
overtrue merged 1 commit into
mainfrom
overtrue/ui-audit-skill

Conversation

@overtrue

Copy link
Copy Markdown
Collaborator

Pull Request

Description

Polishes UI details across auth and dashboard surfaces, covering spacing, alignment, density, accessibility labels, table/form/dialog details, theme handling, and icon sizing. Adds a reusable skills/ui-audit package so future UI walkthroughs can reuse the same inspection matrix. Also repairs the lockfile by removing an accidental extra YAML document so frozen installs can parse it.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Code refactoring
  • Performance improvement
  • Test improvements
  • Security fix

Testing

  • Unit tests added/updated
  • Manual testing completed
CI=true pnpm install --frozen-lockfile
pnpm type-check
pnpm lint
pnpm format:check
git diff --check
python3 /Users/overtrue/.nvm/versions/node/v22.14.0/lib/node_modules/clawdbot/skills/skill-creator/scripts/quick_validate.py skills/ui-audit
node --test tests/lib/i18n-locales.test.js

Full direct node --test tests/lib/*.test.js tests/lib/*.test.ts is blocked by the repository's current test runner setup for TS imports and extension resolution. The direct i18n test passes after adding the missing locale keys.

Checklist

  • Code follows the project's style guidelines
  • Self-review completed
  • TypeScript types are properly defined
  • All commit messages are in English (Conventional Commits)
  • All existing tests pass
  • No new dependencies added, or they are justified

Related Issues

N/A

Screenshots (if applicable)

Screenshots were captured during the local UI walkthrough; no hosted screenshots are attached to this PR body.

Additional Notes

This PR includes the reusable UI audit skill under skills/ui-audit/.

@overtrue overtrue changed the title Fix UI polish details and add audit skill fix: UI polish details and add audit skill Jun 15, 2026
@overtrue overtrue force-pushed the overtrue/ui-audit-skill branch from 2a3ab71 to 9058d44 Compare June 15, 2026 06:05
@overtrue overtrue marked this pull request as ready for review June 15, 2026 06:07
@overtrue overtrue merged commit 8e3161e into main Jun 15, 2026
10 checks passed
@overtrue overtrue deleted the overtrue/ui-audit-skill branch June 15, 2026 06:09
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