Bug #2071
M-ID/0000161: ALT und TITLE bei Smilie-Übersicht
| Status: | Resolved | Start date: | 2007-05-08 | |
|---|---|---|---|---|
| Priority: | -- undefined -- | Due date: | ||
| Assignee: | - | % Done: | 100% |
|
| Category: | Forum | |||
| Target version: | - | |||
| Votes: | 0 |
Description
In class.tx_mmforum_pi1.php wird in der Funktion show_smilie_db() das alt-Tag falsch gesetzt und bleibt deshalb leer. Das title-Tag wird gar nicht gesetzt.
Bisher:
$imgInfo['alt'] = $row['emoticon'];
Sinnvoll:
$imgInfo['alt'] = $row['code'];
$imgInfo['title'] = $row['code'];
Notes:
icy
04-18-08 21:02
$row['code'] macht meiner Meinung nach nur begrenzt Sinn, da ein alt-Tag nicht irgendwas sondern eine im Zusammenhang sinnvolle Beschreibung beinhalten soll.
ein
$imgInfo['alt'] = "Smilie" . $row['code'];
macht mehr Sinn, ist aber auch wenig zufriedenstellend.
Besser wäre es, wenn $row['emoticon'] mit sinnvollen Beschreibungen der Smilies gefüllt würde.
(Verweise auf BITV/ATAG lass ich mal)
mhelmich
04-20-08 12:26
Das 'code'-Feld für den Alt- und Title-Text zu verwenden, macht meiner Ansicht nach keinen Sinn. Eigentlich war ursprünglich das 'emoticon'-Feld eben dafür gedacht, leider enthält dieses Feld in der Datenbank keinen Wert. Erschwerend kommt hinzu, dass dieses Feld unter Umständen auch sprachabhängig sein muss. In einer der nächsten Versionen lass ich mir mal was dazu einfallen.
Associated revisions
[~TASK] The RSS feed feature is now documented in the official maunal (fixed #4720)
[~TASK] The messaging user search script was redesigned (issue #5932)
[~TASK] Cleaned up TCA. Avatar upload now respects TCA setting for max file size (fixes #4129)
[#BUGFIX] ALT- and TITLE tags for smilie buttons are now set correctly (fixes #2071)
[~TASK] The RSS feed feature is now documented in the official maunal (fixed #4720)
[~TASK] The messaging user search script was redesigned (issue #5932)
[~TASK] Cleaned up TCA. Avatar upload now respects TCA setting for max file size (fixes #4129)
[#BUGFIX] ALT- and TITLE tags for smilie buttons are now set correctly (fixes #2071)
git-svn-id: https://svn.typo3.org/TYPO3v4/Extensions/mm_forum/trunk@28560 735d13b6-9817-0410-8766-e36946ffe9aa
History
Updated by Martin Helmich over 3 years ago
- Status changed from Accepted to Resolved
- % Done changed from 0 to 100
Applied in changeset r28560.