Skip to content

Fix parsing of <script> and <style> tags #20

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

Merged
merged 1 commit into from
Aug 30, 2016
Merged

Conversation

remarkablemark
Copy link
Owner

Fixes #19

Tasks:

  • Ensure empty <script> tag is converted to a React element and not an empty array []
  • Add missing logic for parsing <style> DOM nodes to React element
  • Update tests and mocks

- Ensure empty <script> tag is converted to a React element and
  not an empty array `[]`
- Add missing logic for parsing <style> DOM nodes to React element
- Update tests and mocks
@remarkablemark remarkablemark added the bug Something isn't working label Aug 30, 2016
@remarkablemark remarkablemark self-assigned this Aug 30, 2016
@coveralls
Copy link

Coverage Status

Coverage increased (+0.05%) to 89.623% when pulling 9170e92 on bug-script-style into 7522a0a on master.

@remarkablemark remarkablemark merged commit 020973e into master Aug 30, 2016
@remarkablemark remarkablemark deleted the bug-script-style branch August 30, 2016 16:37
remarkablemark added a commit that referenced this pull request Aug 30, 2016
- Fix parsing of <script> and <style> tags (#20)
- Remove `key` parameter in `replace` method and use
  `React.cloneElement` instead
- Update tests and mocks
- Update README and remove `key` parameter in `replace`
- Update ESLint rule for `no-unused-vars`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants