Skip to content

Fix documentation errors: wrong theme colors, missing variables, invalid HTML - #172

Merged
knadh merged 1 commit into
knadh:masterfrom
MADEVAL:fix-docs-consistency
Jun 28, 2026
Merged

Fix documentation errors: wrong theme colors, missing variables, invalid HTML#172
knadh merged 1 commit into
knadh:masterfrom
MADEVAL:fix-docs-consistency

Conversation

@MADEVAL

@MADEVAL MADEVAL commented Jun 28, 2026

Copy link
Copy Markdown
Contributor

customizing.md:

  • Fix 3 wrong color values in Default Oat brown section: danger (#df514c -> #d32f2f), success (#4caf50 -> #008032), warning (#ff8c00 -> #a65b00)
  • Add missing --faint-foreground variable to Default Oat brown section
  • Fix wrong color values in Theming reference section: primary, secondary-foreground, danger, success, warning, ring
  • Remove non-existent --accent-foreground variable from Theming reference
  • Fix dark mode docs: data-theme is not built-in, dark mode uses light-dark() + color-scheme automatically following OS preference

sidebar.md:

  • Fix prose line 56: data-topnav goes on nav element, not header
  • Fix structure table: data-topnav element is nav, not header
  • Fix invalid HTML at line 33: removed duplicate class attribute, changed class=sm to class=small

button.md:

  • Remove non-existent data-variant=primary from variant list. Default button without variant is already primary.

toast.md:

  • Fix default variant in options table: empty string changed to 'info' (matches toast.js line 94)

…lid HTML

customizing.md:
- Fix 3 wrong color values in Default Oat brown section:
  danger (#df514c -> #d32f2f), success (#4caf50 -> #008032),
  warning (#ff8c00 -> #a65b00)
- Add missing --faint-foreground variable
- Fix wrong color values in Theming reference section:
  primary, secondary-foreground, danger, success, warning, ring
- Remove non-existent --accent-foreground variable
- Fix dark mode docs: data-theme is not built-in, dark mode uses
  light-dark() + color-scheme automatically

sidebar.md:
- Fix prose: data-topnav goes on nav, not header
- Fix structure table: element is nav, not header
- Fix invalid HTML: double class attribute, class=sm -> small

button.md:
- Remove non-existent data-variant=primary from docs

toast.md:
- Fix default variant: '' -> 'info' (matches toast.js)
@knadh
knadh merged commit 0696a15 into knadh:master Jun 28, 2026
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