Add support for SvgSpriteIconProvider for SVG assets (support for TYPO3 V14 Themes)

[OlivierSC]

Currently, when icons for content types are in SVG format, the provider used is "SvgIconProvider".

content-blocks/Classes/Service/Icon/ContentTypeIconResolver.php

Line 46 in b816508

$iconProviderClass = $fileExtension === 'svg' ? SvgIconProvider::class : BitmapIconProvider::class;

This provider generates a call to the SVG via an IMG tag.
<img src="path/to/ifile.svg"/>

This type of call doesn't seem to allow, in SVG file, the use of CSS variables offered by the TYPO3 backend, such as:

• currentColor
• icon-color-accent (V14)

Therefore, it doesn't allow for the use of the new themes offered by TYPO3 V14 with different color variations.

Would it be possible to switch to the "SvgSpriteIconProvider" provider, which generates a true SVG tag on the page and allows for the use of these CSS variables?
<svg><use xlink:href="path/to/ifile.svg"></svg>

Additional context

• TYPO3 V14 and "theme" feature

[nhovratov]

@OlivierSC Hi, thanks for the feature request. We can't just switch to SvgSpriteIconProvider, or else every icon will break in the backend, because no sprite is defined for those icons. The SvgIconProvider is able to create "inline" markup, but only if the underlying code requests "alternative markup", which does not happen for the icons of Content Element in the Layout View. In fact, TYPO3 does not have alternative colors for its Core Content Elements. So my suggestion is to provide "nuetral" icon colors, like the grayish ones in TYPO3. They work both in dark and light mode very well.

[sbuerk]

@nhovratov You may possible want to adopt:

https://github.com/web-vision/deepl-base/blob/main/Classes/Imaging/IconProvider/DeeplBaseSvgIconProvider.php

as temporary solution - has it on the list to add it as feature to the core.

E.g. https://github.com/web-vision/deepltranslate-core/blob/0377f623f263e6dc933a6868fc2bf202a786eb84/Configuration/Icons.php#L25-L28

[nhovratov]

@sbuerk Oh, cool! Thanks for the hint. This may be a solution as long as it is candidate for Core.