Skip to content

Conversation

@benmccann
Copy link
Member

I think this makes the docs more approachable. If you're learning about custom elements it's more helpful to have everything on a single page vs split across two pages. And if you're learning about Svelte in general and never using custom elements then you don't need to see $host in the sidebar, which makes it feel like there are more concepts to learn

The one case where this is slightly worse is if you see $host in your codebase and want to know what it does. But you can find it very quickly via search and I'm not sure we really need everything in the sidebar (e.g. we don't have $effect.pre in the sidebar, which is probably used a lot more)

@changeset-bot
Copy link

changeset-bot bot commented Mar 20, 2025

⚠️ No Changeset found

Latest commit: 4285bf9

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@svelte-docs-bot
Copy link

@github-actions
Copy link
Contributor

Playground

pnpm add https://pkg.pr.new/svelte@15562

@Rich-Harris
Copy link
Member

I think it's better the way it is now. It's reference documentation; it would be weird if the 'runes' section didn't contain all the runes

@benmccann
Copy link
Member Author

Yeah, fair enough. I might take a stab at separating it into more of a guide vs reference like we have in the SvelteKit docs

@benmccann benmccann closed this Mar 21, 2025
@Rich-Harris
Copy link
Member

Anecdotally I've heard people say that the SvelteKit docs are trash compared to the Svelte docs, so we would want to be extremely cautious before embarking on something like that

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants