Website cleanup: remove docs, cookies, accessibility fixes, package upgrades#547
Merged
Website cleanup: remove docs, cookies, accessibility fixes, package upgrades#547
Conversation
Collaborator
MaanavD
commented
Mar 23, 2026
- Remove docs section (/docs routes, content, sidebar, doc components)
- Remove sidebar:state cookie
- Add Microsoft Privacy Statement link to footer
- Accessibility fixes (WCAG compliance): aria-hidden on decorative icons, new tab warnings on external links, role=status/alert on loading/error, semantic main landmark, nav landmarks in footer, aria-busy on skeletons
- Upgrade packages to latest compatible versions
- Migrate lucide-svelte v1 (Github brand icon to inline SVG)
- Remove unused packages (shiki, flexsearch, mdsvex, adapter-cloudflare, adapter-static)
- Clean up svelte.config.js and utils.ts
…pgrades - Remove docs section (/docs routes, content, sidebar, doc components) - Remove sidebar:state cookie - Add Microsoft Privacy Statement link to footer - Accessibility fixes (WCAG compliance): aria-hidden on decorative icons, new tab warnings on external links, role=status/alert on loading/error, semantic main landmark, nav landmarks in footer, aria-busy on skeletons - Upgrade packages to latest compatible versions - Migrate lucide-svelte v1 (Github brand icon to inline SVG) - Remove unused packages (shiki, flexsearch, mdsvex, adapter-cloudflare, adapter-static) - Clean up svelte.config.js and utils.ts
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
There was a problem hiding this comment.
Pull request overview
This PR removes the in-site documentation system (routes, content, sidebar/navigation/search, and supporting utilities) from the www/ SvelteKit site, while also applying a set of WCAG-oriented accessibility improvements across the remaining UI and upgrading/removing related packages.
Changes:
- Removed
/docsroutes, markdown content pipeline (mdsvex), and all docs UI components (renderer, TOC, search, nav, sidebar). - Added multiple accessibility improvements (decorative icons
aria-hidden, semantic landmarks, loading/error roles, “opens in new tab” labeling). - Upgraded
www/dependencies and migratedlucide-svelteGitHub icon usage to inline SVG.
Reviewed changes
Copilot reviewed 69 out of 70 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| www/svelte.config.js | Removes mdsvex preprocessing and docs-related prerender error handling. |
| www/src/routes/models/components/ModelFilters.svelte | Adds a11y labels and hides decorative icons from AT. |
| www/src/routes/models/components/ModelDetailsModal.svelte | Adds aria-hidden on icons and new-tab labeling on external link. |
| www/src/routes/models/components/ModelCardSkeleton.svelte | Marks skeleton card as busy with a loading label. |
| www/src/routes/models/components/ModelCard.svelte | Adds aria-hidden to decorative icons and new-tab labeling for license link. |
| www/src/routes/models/+page.svelte | Adds role=status/role=alert and hides spinner/icon for a11y. |
| www/src/routes/docs/[...slug]/+page.ts | Removes docs page loader. |
| www/src/routes/docs/[...slug]/+page.svelte | Removes docs page renderer usage. |
| www/src/routes/docs/+page.ts | Removes docs index loader. |
| www/src/routes/docs/+page.svelte | Removes docs index renderer usage. |
| www/src/routes/docs/+layout.ts | Removes docs layout loader (nav/search index generation). |
| www/src/routes/docs/+layout.svelte | Removes docs layout UI (sidebar/header/search). |
| www/src/routes/+error.svelte | Uses semantic <main> and hides decorative icons from AT. |
| www/src/lib/utils.ts | Removes docs-related helpers (slug/getDoc/findMatch) and unused imports. |
| www/src/lib/types/docs.ts | Deletes docs-specific type definitions. |
| www/src/lib/components/ui/sidebar/sidebar.svelte | Deletes sidebar UI implementation. |
| www/src/lib/components/ui/sidebar/sidebar-trigger.svelte | Deletes sidebar trigger component. |
| www/src/lib/components/ui/sidebar/sidebar-separator.svelte | Deletes sidebar separator component. |
| www/src/lib/components/ui/sidebar/sidebar-rail.svelte | Deletes sidebar rail component. |
| www/src/lib/components/ui/sidebar/sidebar-provider.svelte | Deletes sidebar provider (including cookie persistence). |
| www/src/lib/components/ui/sidebar/sidebar-menu.svelte | Deletes sidebar menu component. |
| www/src/lib/components/ui/sidebar/sidebar-menu-sub.svelte | Deletes sidebar submenu wrapper. |
| www/src/lib/components/ui/sidebar/sidebar-menu-sub-item.svelte | Deletes sidebar submenu item. |
| www/src/lib/components/ui/sidebar/sidebar-menu-sub-button.svelte | Deletes sidebar submenu link/button. |
| www/src/lib/components/ui/sidebar/sidebar-menu-skeleton.svelte | Deletes sidebar menu skeleton. |
| www/src/lib/components/ui/sidebar/sidebar-menu-item.svelte | Deletes sidebar menu item wrapper. |
| www/src/lib/components/ui/sidebar/sidebar-menu-button.svelte | Deletes sidebar menu button + tooltip behavior. |
| www/src/lib/components/ui/sidebar/sidebar-menu-badge.svelte | Deletes sidebar menu badge. |
| www/src/lib/components/ui/sidebar/sidebar-menu-action.svelte | Deletes sidebar menu action button. |
| www/src/lib/components/ui/sidebar/sidebar-inset.svelte | Deletes sidebar inset layout component. |
| www/src/lib/components/ui/sidebar/sidebar-input.svelte | Deletes sidebar input component. |
| www/src/lib/components/ui/sidebar/sidebar-header.svelte | Deletes sidebar header component. |
| www/src/lib/components/ui/sidebar/sidebar-group.svelte | Deletes sidebar group component. |
| www/src/lib/components/ui/sidebar/sidebar-group-label.svelte | Deletes sidebar group label component. |
| www/src/lib/components/ui/sidebar/sidebar-group-content.svelte | Deletes sidebar group content wrapper. |
| www/src/lib/components/ui/sidebar/sidebar-group-action.svelte | Deletes sidebar group action button. |
| www/src/lib/components/ui/sidebar/sidebar-footer.svelte | Deletes sidebar footer component. |
| www/src/lib/components/ui/sidebar/sidebar-content.svelte | Deletes sidebar content wrapper. |
| www/src/lib/components/ui/sidebar/index.ts | Deletes sidebar public exports barrel. |
| www/src/lib/components/ui/sidebar/context.svelte.ts | Deletes sidebar context/state implementation. |
| www/src/lib/components/ui/sidebar/constants.ts | Deletes sidebar constants (incl. sidebar:state cookie name). |
| www/src/lib/components/social-media.svelte | Updates external link labeling + hides decorative SVG from AT. |
| www/src/lib/components/sidebar-page.svelte | Removes an example page built around the sidebar layout. |
| www/src/lib/components/search-bar.svelte | Removes docs search UI (Command dialog + keyboard shortcut). |
| www/src/lib/components/install-command.svelte | Improves icon accessibility with role=img/aria-hidden. |
| www/src/lib/components/home/nav.svelte | Removes redundant SR-only label (keeps aria-label), hides decorative chevron. |
| www/src/lib/components/home/hero.svelte | Updates external docs button a11y label to indicate new tab. |
| www/src/lib/components/home/footer.svelte | Adds footer nav landmarks, new-tab labels, and Privacy Statement link. |
| www/src/lib/components/home/features.svelte | Marks decorative icons as aria-hidden. |
| www/src/lib/components/download-dropdown.svelte | Removes lucide GitHub icon import and uses inline GitHub SVG. |
| www/src/lib/components/document/toc.svelte.ts | Removes docs table-of-contents logic. |
| www/src/lib/components/document/table-of-contents.svelte | Removes docs TOC sidebar component. |
| www/src/lib/components/document/promo-card.svelte | Removes docs promo card. |
| www/src/lib/components/document/mobile-table-of-contents.svelte | Removes docs mobile TOC component. |
| www/src/lib/components/document/doc-renderer.svelte | Removes docs renderer (including Shiki integration). |
| www/src/lib/components/document/doc-header.svelte | Removes docs header component. |
| www/src/lib/components/document/doc-content.svelte | Removes docs content component (anchors + Shiki highlighting/copy). |
| www/src/lib/components/doc-search.svelte.ts | Removes docs search indexing (FlexSearch). |
| www/src/lib/components/doc-navigation.svelte.ts | Removes docs navigation generation logic. |
| www/src/lib/components/breadcrumb.svelte | Removes breadcrumb component previously used by docs layout. |
| www/src/lib/components/app-sidebar.svelte | Removes app sidebar that rendered generated docs nav. |
| www/src/content/styling/theme.md | Removes docs content markdown. |
| www/src/content/styling/syntax-highlighting.md | Removes docs content markdown. |
| www/src/content/models.md | Removes docs content markdown. |
| www/src/content/index.md | Removes docs content markdown. |
| www/src/content/features.md | Removes docs content markdown. |
| www/src/content/examples.md | Removes docs content markdown. |
| www/src/content/customize.md | Removes docs content markdown. |
| www/package.json | Upgrades dependencies and removes docs-related packages (mdsvex/shiki/flexsearch, etc.). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Collaborator
Author
|
@copilot open a new pull request to apply changes based on the comments in this thread |
Contributor
…al links (#548) Svelte does not interpolate `{expr}` inside quoted attribute strings — they are treated as literals. Four `aria-label` attributes were broken in this way, and one external-link label was missing the "opens in new tab" annotation required for WCAG consistency. ## Changes - **`ModelDetailsModal.svelte` / `ModelCard.svelte`**: Change `aria-label="{model.license} ..."` → `aria-label={`...${model.license}...`}` so the actual license value reaches assistive tech instead of the literal string `{model.license}`. - **`social-media.svelte`**: Same fix for `{title}` — `aria-label="{title} (opens in new tab)"` → `aria-label={`${title} (opens in new tab)`}`. - **`download-dropdown.svelte`**: Add `(opens in new tab)` to the GitHub Releases link `aria-label`, consistent with all other `target="_blank"` links in the codebase. ```svelte <!-- Before: Svelte treats {model.license} as a literal string --> aria-label="{model.license} license (opens in new tab)" <!-- After: properly interpolated --> aria-label={`${model.license} license (opens in new tab)`} ``` <!-- START COPILOT CODING AGENT TIPS --> --- 🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. [Learn more about Advanced Security.](https://gh.io/cca-advanced-security) --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: MaanavD <24942306+MaanavD@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.