akiran / akiran/react-highlight

innerHTML breaks syntax highlighting

Ouverte
#85 3 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
Langage dominant
JavaScript
Étoiles
761
Forks
82
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

I'm on nodejs 18.3, so I'm wondering if it's a compatibility thing, but here's a [codesandbox](https://codesandbox.io/s/react-highlight-forked-1ms6sr?file=/src/index.jsl) that demonstrates it.

I was able to make it to work by hacking [here](https://github.com/akiran/react-highlight/blob/master/src/index.js#L34) to do this:
```
if (innerHTML) {
var wrapped="

" +  children + "
";
props.dangerouslySetInnerHTML = { __html: wrapped };
...
```

This is because [this function](https://github.com/akiran/react-highlight/blob/master/src/index.js#L17) doesn't find the element thanks to lack of `

` wrapper:

```
highlightCode() {
const nodes = this.el.querySelectorAll('pre code');

for (let i = 0; i < nodes.length; i++) {
hljs.highlightBlock(nodes[i])
}
}
```

I'd do a PR myself but i'm a novice at this js and react stuff and i'm pretty sure my hack isn't the right way to fix it. If you want to explain what I need to do I can submit the PR.

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.