Feeling Lucky
?
[#29094] - Add alt and aria-hidden="true" for logo
- Closed
- 14 May 2020
- Medium
- Build: 4.0-dev
- # 29094
- Diff
- DenysNosov:patch-8
- Pending Hound Hound is busy sniffing around... Details
User tests: Successful: Unsuccessful:
Summary of Changes
Fix image (logo) for screen reader.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration Templates (admin) |
DenysNosov
- comment
- 14 May 2020
Why read the logo when logged in?
brianteeman
- comment
- 14 May 2020
Sorry I do not understand that comment
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2020-05-14 11:33:17 |
| Closed_By | ⇒ | Quy | |
| Labels |
Added:
?
|
||
Sorry but this is not correct.
Adding aria-hidden=true removes the image from the accessibility tree so it would not matter what the alt description was as a screen reader would never see it.
A decorative image should not have an alt description - it is absolutely correct for it to have the current alt="" in this use case
There is however a bug in that the template style has an option for an alt description to be entered but it is never used. Personally I would be removing that option.