perf: lazy load toast-ui editor dependencies

[JammingBen]

Description

Dynamically load toast-ui editor languages and the syntax highlight plugin for markdown files. This improves editor loading times in other scenarios where of one these 2 things are not needed.

Related Issue

• Fixes Prevent toast-ui editor from always loading all dependencies #10982

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
• Technical debt
• Tests
• Documentation
• Maintenance (e.g. dependency updates or tooling)

[AlexAndBear]

Looks a little clumsy but I fear that's the downside of making it performant 😁

[JammingBen]

Looks a little clumsy but I fear that's the downside of making it performant 😁

Totally agree :( Also the weird dist-imports of the lib contribute to that unfortunately.

[kulmann]

Looks a little clumsy but I fear that's the downside of making it performant 😁

Totally agree :( Also the weird dist-imports of the lib contribute to that unfortunately.

Do you think this is something to solve or propose upstream? ;-)

[AlexAndBear]

Could you think of adding a new ts file for the translations where you import the lang files and then just importing that file Async, for now ?

[kulmann]

Could you think of adding a new ts file for the translations where you import the lang files and then just importing that file Async, for now ?

good idea 👍

[JammingBen]

Do you think this is something to solve or propose upstream? ;-)

I didn't dig too deep, but it seems that you can use the non-dist imports, but then you have to explicitly name the supported languages for the syntax highlighting.

Could you think of adding a new ts file for the translations where you import the lang files and then just importing that file Async, for now ?

good idea 👍

Hmm I don't see much benefit tbh. Also it would raise the question where this file would live. We're not in an app context here, this is just a single web-pkg component. But if you 2 want to have it that way, sure that's possible 🤷

[AlexAndBear]

Just decluttering I guess. Could make a new Folder
/Components/TextEditor/editor.vue
/Components/TextEditor/i18n.ts

[sonarcloud]

Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud