Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add support for unicode characters in anchor links #328

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

## Uh, oh

There is no section named [Potato](#potato).

There is an anchor named [Tomato](#tomato).

## Header with special char ✨
## Header with special char at end ✨
Copy link

@chris-grabcad chris-grabcad Oct 2, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dklimpel I see there are quite a few bugs related to this, I have some real world examples you might want to try too - as they actively error for me in 3.12 (from megalinter)

## --docker
[--docker](#--docker)

## Step 7 - Lint & Test
[Step 7 - Lint \& Test](#step-7---lint--test)

## Product Owner / Design Approval
[Product Owner / Design Approval](#product-owner--design-approval)

These are as they are created by markdown all in one extension in vscode

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have this added to the tests. And it works.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's great news, so this fix isn't in 3.12 - and just needs merging and releasing?

Do we know when it will be released?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tcort any ideas for a timeline?

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