Skip to content

Minor Dialog and dropdown style tweaks#797

Merged
netchampfaris merged 2 commits into
mainfrom
fix/dialog-menu-styles
Jun 23, 2026
Merged

Minor Dialog and dropdown style tweaks#797
netchampfaris merged 2 commits into
mainfrom
fix/dialog-menu-styles

Conversation

@netchampfaris

@netchampfaris netchampfaris commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Two small styling tweaks:

  • Dialog: title is now text-2xl-semibold / text-ink-gray-8 (was text-3xl-semibold / text-ink-gray-9) — smaller and softer.
  • Menu (dropdown): highlighted item background uses surface-alpha-gray-2 instead of surface-gray-2.

🤖 Generated with Claude Code

Docs preview: https://ui.frappe.io/pr-preview/pr-797/

Coverage: 57.15% (±0.00% vs main)

netchampfaris and others added 2 commits June 24, 2026 02:26
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@barista-for-frappe

barista-for-frappe Bot commented Jun 23, 2026

Copy link
Copy Markdown

Looks good — two cosmetic token tweaks, no public API change.

  • Dialog.vue:78 — smaller/softer title (text-2xl-semibold / text-ink-gray-8); both tokens are used elsewhere (e.g. Card.vue:8).
  • Menu/utils.ts:162 — highlighted item now uses surface-alpha-gray-2; confirmed it's a real semantic token (surface-alpha/gray-2gray-alpha.100). Safe.

barista · claude-opus-4-8 · 9.2k in / 6.0k out · 1828k cached · 97s · $0.920

@netchampfaris netchampfaris added the beta-release Auto-publish a beta to npm when the PR is merged label Jun 23, 2026
@netchampfaris netchampfaris merged commit cf6527d into main Jun 23, 2026
6 of 7 checks passed
@netchampfaris netchampfaris deleted the fix/dialog-menu-styles branch June 23, 2026 21:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

beta-release Auto-publish a beta to npm when the PR is merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant