You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Including pat-tooltip includes pat-markdown which includes pat-syntax-highlight.
That includes the whole highlight.js library for no good reason. We do not need pat-markdown nor pat-syntax-highlight in pat-tiptap. The libraries are imported dynamically and only when needed, so there is no runtime issue with that. But the compiled bundle size is quite big and if we want to include the compiled bundle via npm packages so that these packages can be used via CDNs right away, this is just a waste of space.
Find a way to register pat-markdown to pat-tooltip if pat-markdown and not the other way around.
This would solve the issue.
Also, check the situation if hightlight.js / Webpack supports dynamic imports. Currently it doesn't. See: highlightjs/highlight.js#3634
Analyze bundle with webpack-bundle-analyzer.
The text was updated successfully, but these errors were encountered:
Including pat-tooltip includes pat-markdown which includes pat-syntax-highlight.
That includes the whole highlight.js library for no good reason. We do not need pat-markdown nor pat-syntax-highlight in pat-tiptap. The libraries are imported dynamically and only when needed, so there is no runtime issue with that. But the compiled bundle size is quite big and if we want to include the compiled bundle via npm packages so that these packages can be used via CDNs right away, this is just a waste of space.
Find a way to register pat-markdown to pat-tooltip if pat-markdown and not the other way around.
This would solve the issue.
Also, check the situation if hightlight.js / Webpack supports dynamic imports. Currently it doesn't. See:
highlightjs/highlight.js#3634
Analyze bundle with
webpack-bundle-analyzer
.The text was updated successfully, but these errors were encountered: