Skip to content

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

Merged
remarkablemark merged 1 commit intomasterfrom
bug-script-style
Aug 30, 2016
Merged

Fix parsing of <script> and <style> tags#20
remarkablemark merged 1 commit intomasterfrom
bug-script-style

Conversation

@remarkablemark
Copy link
Copy Markdown
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
Copy Markdown

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