Skip to content
Snippets Groups Projects
Commit 999fdc02 authored by Oliver Klee's avatar Oliver Klee Committed by Georg Ringer
Browse files

[BUGFIX] Use empty alt text for EM extension icons

For images that cannot be loaded (e.g., when the extension is outside
the document root), the extension name will be printed twice, which
looks ugly and also breaks the layout.

As the extension icon is purely decorative, the alt attribute should
be an empty string.

Resolves: #91031
Releases: master, 9.5
Change-Id: I71be7978cc3245f127d864586dd5d54c1a6aae10
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/64167


Tested-by: default avatarTYPO3com <noreply@typo3.com>
Tested-by: default avatarBjörn Jacob <bjoern.jacob@tritum.de>
Tested-by: default avatarChristian Eßl <indy.essl@gmail.com>
Tested-by: default avatarOliver Bartsch <bo@cedev.de>
Tested-by: default avatarJosef Glatz <josefglatz@gmail.com>
Tested-by: default avatarGeorg Ringer <georg.ringer@gmail.com>
Reviewed-by: default avatarBjörn Jacob <bjoern.jacob@tritum.de>
Reviewed-by: default avatarChristian Eßl <indy.essl@gmail.com>
Reviewed-by: default avatarOliver Bartsch <bo@cedev.de>
Reviewed-by: default avatarJosef Glatz <josefglatz@gmail.com>
Reviewed-by: default avatarGeorg Ringer <georg.ringer@gmail.com>
parent 6541b3eb
No related merge requests found
......@@ -4,7 +4,7 @@
<div class="distribution-detail">
<div class="distribution-detail-previewpane">
<em:distributionImage extensionkey="{extension.extensionKey}" alt="{extension.title}" height="225" width="300" />
<em:distributionImage extensionkey="{extension.extensionKey}" alt="" height="225" width="300" />
</div>
<div class="distribution-detail-body">
<div class="distribution-detail-header">
......
......@@ -96,7 +96,7 @@
</td>
<td>
<f:if condition="{extension.icon}">
<img class="extension-icon" src="{extension.icon}" alt="{extension.title}" />
<img class="extension-icon" src="{extension.icon}" alt="" />
</f:if>
<span title="{extension.description}">{extension.title}</span>
</td>
......
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment