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

Full-featured text extraction from the first H1 heading #3578

Merged
merged 2 commits into from Mar 8, 2024
Merged

Conversation

oprypin
Copy link
Contributor

@oprypin oprypin commented Feb 24, 2024

This partly reverts changes in commit e755aae as some of that functionality will be deprecated.
Several more edge cases are taken into account now.

It now more closely follows the approach that Python-Markdown will take soon.

@waylan This contains code copied from your pull request. (The purpose of copying is to not require a new version of Python-Markdown.) Are you OK with this code being put under the license of this repository?

This partly reverts changes in commit e755aae as some of that functionality will be deprecated.
Several more edge cases are taken into account now.

Co-authored-by: Waylan Limberg <waylan.limberg@icloud.com>
@oprypin oprypin requested a review from waylan February 24, 2024 11:39
@oprypin oprypin changed the title Follow Python-Markdown approach for getting the title from the first H1 Full-featured text extraction from the first H1 heading Feb 24, 2024
@oprypin
Copy link
Contributor Author

oprypin commented Feb 24, 2024

I have now also implemented extraction of alt attributes of image tags

@oprypin
Copy link
Contributor Author

oprypin commented Mar 1, 2024

@waylan , I don't need much of a review here, but I would need to hear from you whether you're OK to include the source code that's derived from your code but under the license of this repository.

Copy link
Member

@waylan waylan left a comment

Choose a reason for hiding this comment

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

Looks good to me. I approve the use of the borrowed code. 👍

@oprypin oprypin merged commit 4ad084e into master Mar 8, 2024
34 checks passed
@oprypin oprypin deleted the toc2 branch March 8, 2024 20:11
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.

Header and navigation sidebars display incorrectly arbitrary markup
2 participants