Skip to content

Conversation

@garikAsplund
Copy link
Contributor

Before submitting the PR, please make sure you do the following

  • It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
  • Prefix your PR title with feat:, fix:, chore:, or docs:.
  • This message body should clearly illustrate what problems it solves.
  • Ideally, include a test that fails without this PR but passes with it.
  • If this PR changes code within packages/svelte/src, add a changeset (npx changeset).

Tests and linting

  • Run the tests with pnpm test and lint the project with pnpm lint

Update arrows to Unicode for looks

Hi, sorry if I didn't submit an issue first--I think this is a small enough change that it's not exactly warranted. I just wanted to switch to nicer looking arrows for aesthetic reasons.

From this:
Screenshot 2025-03-16 at 11 35 49 AM

To this:
Screenshot 2025-03-16 at 11 32 44 AM

Other bug?

Somewhat related, I'm baffled over why this code outputs the way it does. The code seems correct, but it still adds the 'count' when it shouldn't. Found in the same file. I know it's a minor thing, but still 🫤

Screenshot 2025-03-16 at 11 33 08 AM Screenshot 2025-03-16 at 11 21 03 AM

@svelte-docs-bot
Copy link

@Rich-Harris
Copy link
Member

thank you! this looks way better. yeah, the diff syntax is finicky and brittle. there's a few tweaks we could make to this document to make it look a little better, but in some cases it's just a bit buggy and can't really be helped

@Rich-Harris Rich-Harris merged commit f30d75a into sveltejs:main Mar 16, 2025
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