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

Fix sup summary #2008

Merged
merged 1 commit into from
Oct 30, 2022
Merged

Fix sup summary #2008

merged 1 commit into from
Oct 30, 2022

Conversation

mscherer
Copy link
Contributor

@mscherer mscherer commented Oct 26, 2022

If the summary contain HTML using <sup> like "the 1<sup>st<sup>", it get filtered as the regexp that remove footnotes is not precise enough to remove just the footnotes.

IMPORTANT: Please do not create a Pull Request adding a new feature without discussing it first.

The place to discuss new features is the forum: https://zola.discourse.group/
If you want to add a new feature, please open a thread there first in the feature requests section.

Sanity check:

  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Code changes

(Delete or ignore this section for documentation changes)

  • Are you doing the PR on the next branch?

@mscherer mscherer marked this pull request as draft October 26, 2022 19:58
If the summary contain HTML using <sup> like "the 1<sup>st</<sup>", it get
filtered as the regexp that remove footnotes is not precise
enough to remove just the footnotes.
@mscherer mscherer marked this pull request as ready for review October 26, 2022 20:05
@Keats Keats merged commit e3bf9ac into getzola:next Oct 30, 2022
@Keats
Copy link
Collaborator

Keats commented Oct 30, 2022

Thanks!

Keats pushed a commit that referenced this pull request Feb 16, 2023
If the summary contain HTML using <sup> like "the 1<sup>st</<sup>", it get
filtered as the regexp that remove footnotes is not precise
enough to remove just the footnotes.
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