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

Add inline element to handle script attribute. #27

Merged
merged 2 commits into from Oct 17, 2019
Merged

Add inline element to handle script attribute. #27

merged 2 commits into from Oct 17, 2019

Conversation

gfaugere
Copy link
Contributor

Simple InlineListener to add support the script attribute and the generated tags (sup and sub).

Please let me know if there are issues with the way I wrote the class or anything!

@nadar nadar added the enhancement New feature or request label Oct 17, 2019
Copy link
Owner

@nadar nadar left a comment

Choose a reason for hiding this comment

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

Thank you for helping us making the library better. In general this looks good, i just have some small annotations.

Thank you!

CHANGELOG.md Outdated Show resolved Hide resolved
src/listener/Script.php Outdated Show resolved Hide resolved
src/listener/Script.php Show resolved Hide resolved
src/listener/Script.php Show resolved Hide resolved
@codeclimate
Copy link

codeclimate bot commented Oct 17, 2019

Code Climate has analyzed commit dbb67c1 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 99.6%.

View more on Code Climate.

@gfaugere gfaugere requested a review from nadar October 17, 2019 12:48
@nadar nadar merged commit 5bab864 into nadar:master Oct 17, 2019
@nadar
Copy link
Owner

nadar commented Oct 17, 2019

Thanks for the improvement @gfaugere

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants