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

Phase 1 of #24

Merged
merged 2 commits into from
Jun 17, 2024
Merged

Phase 1 of #24

merged 2 commits into from
Jun 17, 2024

Conversation

bowman2001
Copy link
Contributor

@bowman2001 bowman2001 commented Jun 16, 2024

Implements a new delete extension that allows to avoid the possible conflict of the extra subscript extension with the new strike-through extension. This issue is discussed in gohugoio/hugo#12597.

Also adopts two changes in the original Goldmark code for strike-through.

…ating markers > 2 as Goldmark 1.7.2.

This new implementation changes a few details for all inline tags that expect two runes as the marker.

Especially the combination `~~CO~2~~~` won't be rendered as expected.
@jmooring
Copy link
Member

@bowman2001 Thank you. I will try to take this for a spin either later today or early tomorrow.

@jmooring
Copy link
Member

@bowman2001 This looks good. I carefully examined the test cases you removed, and I'm fine with the results. I'll will take care of the Hugo side of things and update the documentation.

@jmooring jmooring merged commit 4d4fcd0 into gohugoio:main Jun 17, 2024
12 checks passed
@jmooring
Copy link
Member

@bowman2001 Thanks again for reacting so quickly to the upstream change. Much appreciated.

@bowman2001
Copy link
Contributor Author

You're welcome! And thanks for all your good work. I've learned a lot, again.

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.

None yet

2 participants