Replace fake SVG icon to proper vector graphics

SVG icon reworked to use no raster data only vector graphics. Sizing fitted to other similar letter-like symbols on the WYSIWYG toolbar:

Hi balu rrtl,

It is true that the default icon does not have the correct formatting.
I see your patch. But after analyse the 'ckeditor5_plugins' directory, I don't see any 'drupalentity' directory in my environment.

My site is built on D9.5.11.

Thanks for your reply.

Hi @balu rrtl, I have tried the shared patch - "MR !41 mergeable" on the Drupal 10.3 version. The patch was applied successfully but the before and after results are the same for me, Attaching images below- Please let me know in case I have missed any steps here.

@Sandeep_k → Thanks for taking the time for testing. I'd test with the following steps:

1. Drush uninstall any previous versions of the module
2. Check manually in the file system the old icon does not remain somewhere
3. Drush cache clear
4. Composer require the MR's branch as package
5. Drush install the module
6. Check in my web browser's DevTool inspector whether the SVG markup of the "E" icon is the same as in the MR.

“The patch was applied successfully”

This sentence makes less and less sense as our community continuously moves over to MR-based workflow. Therefore I'd suggest you not to mention this sentence anymore for any contrib issue which already has a MR. GitLab ensures whether the code changes are applicable or not – no human check is needed for that.

Test failure on the latest pipeline for me seems unrelated to the code changes here:

PHP Fatal error:  Cannot make static method Drupal\Tests\ckeditor5\Kernel\SmartDefaultSettingsTest::provider() non static in class Drupal\Tests\entity_embed\Kernel\UpgradePathTest in /builds/issue/entity_embed-3460832/tests/src/Kernel/UpgradePathTest.php on line 136