Support SVG Favicons

[lzlrd]

How to use GitHub

• Please use the 👍 reaction to show that you are interested into the same feature.
• Please don't comment if you have no relevant information to add. It's just extra noise for everyone subscribed to this issue.
• Subscribe to receive notifications on status change and new comments.

Is your feature request related to a problem? Please describe.
SVG Favicons are supported by a wide variety of browsers. As with the other images supported by the Theming app, it would be preferable to avoid converting SVG images used as favicons and instead leaving them as is.

Describe the solution you'd like
SVG Favicons to not be converted and instead used directly.

Describe alternatives you've considered
n/a

Additional context
Features such as light/dark theme support require SVG images to utilise CSS' prefers-color-scheme feature.

[Salvador1313]

Спасибо.

[Salvador1313]

Спасибо

[solracsf]

For reference https://caniuse.com/link-icon-svg

[lzlrd]

For reference https://caniuse.com/link-icon-svg

If support is a concern, there's no harm in generating an favicon.ico and placing it at the root directory which is the recommended method of a fallback (as described by https://html.spec.whatwg.org/multipage/links.html#rel-icon).

[skjnldsv]

server/apps/theming/lib/ImageManager.php

Lines 329 to 332 in cf1fae2

	if ($key !== 'favicon' || $this->shouldReplaceIcons() === true) {
	$supportedFormats[] = 'image/svg+xml';
	$supportedFormats[] = 'image/svg';
	}

[joshtrichards]

Related: #36607

[hariprabhu571]

Hi, I would like to work in this issue, can you assign this to me?