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 trailing text issue on transformTag #521

Merged
merged 3 commits into from
Dec 7, 2021

Conversation

alex-rantos
Copy link

@alex-rantos alex-rantos commented Dec 4, 2021

Fixes issue #506

Fixes regression caused by #395 in which addedText was never reseted to false after the ontext function was called for that tag.

Solution: reset addedText boolean to false in the onclosetag function.

index.js Show resolved Hide resolved
@alex-rantos alex-rantos changed the title Fix trailing text issue Fix trailing text issue on transformTag Dec 4, 2021
@abea abea self-requested a review December 6, 2021 15:02
Copy link
Contributor

@abea abea left a comment

Choose a reason for hiding this comment

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

This looks great. Thank you. Please add a CHANGELOG entry under an ## Unreleased heading. Include a message like "Thanks to Alexandros Rantos for the contribution." (linking to your Github account if you'd like)

@alex-rantos
Copy link
Author

Thanks for review @abea ! Added a changelog entry - feel free to change the wording if needed.

@abea abea self-requested a review December 7, 2021 16:14
@abea abea merged commit d077c9f into apostrophecms:main Dec 7, 2021
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