Skip to content

fix(ui,ui-themes,ui-menu): Small fixes for doc#2485

Draft
git-nandor wants to merge 2 commits intomasterfrom
small_fixes_for_doc
Draft

fix(ui,ui-themes,ui-menu): Small fixes for doc#2485
git-nandor wants to merge 2 commits intomasterfrom
small_fixes_for_doc

Conversation

@git-nandor
Copy link
Contributor

@git-nandor git-nandor commented Mar 20, 2026

Changes

  • Export light and dark themes from ui meta package

  • Resolve duplicate document ID warnings for multi-version components:

The version map was tracking a flat list of component versions per package
(e.g. ['v1', 'v2']) rather than a per-component mapping. In packages where
export files reference multiple component directories at different versions
(e.g. DateInput/v2 and DateInput2/v1), both v1 and v2 would pass the
inclusion filter for any library version, causing duplicate document ID
warnings when docs were built.

Fix by changing componentVersions to componentDirVersions
(Record<string, string>), mapping each component directory name to its
version. processFile.mts now also extracts componentDirName from the file
path, and isDocIncludedInVersion uses it to look up the exact version for
that specific component rather than checking the whole package.

@git-nandor git-nandor self-assigned this Mar 20, 2026
@github-actions
Copy link

github-actions bot commented Mar 20, 2026

PR Preview Action v1.8.1

QR code for preview link

🚀 View preview at
https://instructure.design/pr-preview/pr-2485/

Built to branch gh-pages at 2026-03-23 08:56 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

@git-nandor git-nandor changed the title fix(ui,ui-themes,ui-menu): remove outdated todo comments and fix wron… fix(ui,ui-themes,ui-menu): Small fixes for doc Mar 23, 2026
@git-nandor git-nandor force-pushed the small_fixes_for_doc branch from a905db2 to 95ed742 Compare March 23, 2026 08:51
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