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: replaced SyntaxHighlighter to prism to fix react components #76

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

idoo
Copy link
Contributor

@idoo idoo commented Jul 19, 2019

looks like language attribute broke it #59 — dropped it to fix highlight

@Bogdan-Lyashenko
Copy link
Owner

@idoo this newer version seems to have significant performance drop, it hangs on few sec when I open Code tab, see profiler below
image

@idoo
Copy link
Contributor Author

idoo commented Jul 26, 2019

@Bogdan-Lyashenko could you show screen how it worked before?

@Bogdan-Lyashenko
Copy link
Owner

@idoo it was smooth :) i.e. it wasn't the issue, now it's quite visible (without even throttling or anything) that it's delays for few sec before render. Not sure what we should do, let's wait with this.
image

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