Icons in @ckeditor/ckeditor5-core should be tree-shake-able

[bendemboski]

✔️ Expected result

When building/bundling CKEditor, only the icons that are needed end up in the bundle

❌ Actual result

All icons end up in the bundle

❓ Possible solution

Because the icons are exported as properties of a single object rather than as individual symbols, they are not tree-shake-able, and anytime any icon is used by any plugin, all the icons are included in the bundle.

📃 Other details

With CKEditor 44, I'm seeing those icons take up ~31k in my bundle, which is up from ~15k in CKEditor 38. So this isn't a massive bundle size impact, but it's still significant, and especially as CKEditor is likely to continue adding features and therefore icons, it will probably continue to grow. Exporting the icons as individual symbols would allow them to be tree-shaken, and make this a non-issue.

---

If you'd like to see this fixed sooner, add a 👍 reaction to this post.

[Witoso]

In reality, this is a bit more complex as we will also need to work on the icons' replacement in the very near future (#16546), but we will keep this in mind (cc @filipsobol @pomek)

[filipsobol]

This was fixed in #17750 (more info: #16546 (comment)) and will be available at the end of Q1.