[5.2] Email alt text on contact

[brianteeman]

Pull Request for Issue #37442 .

Summary of Changes

The wrong language string was being referenced.
This had previously been fixed when using icons but was missed when using images

Comments about using alt titles and possibly invalid DT elements when using images instead of icons is beyond the scope of this PR. That is a bigger issue for the a11y team to consider

Testing Instructions

• Go to Global Configuration -> Component -> Contacts -> Icon panel
• Set Settings drop-down list to Icons
• Select any picture for Custom Email Icon
• Create a contact, add every contact datas (name, address, email, etc.)
• Display the created contact using any kind of template (preferable Cassiopeia) which doesn't use template override for the file "components/com_contact/tmpl/contact/default_address.php"
• Check the alt text of the selected image of email

Actual result BEFORE applying this Pull Request

The image used for the address image has a non translated value of alt="COM_CONTACT_EMAIL"

Expected result AFTER applying this Pull Request

All the images have an ALT text description

Link to documentations

Please select:

• Documentation link for docs.joomla.org:

• No documentation changes for docs.joomla.org needed

• Pull Request link for manual.joomla.org:

• No documentation changes for manual.joomla.org needed

[fgsw]

I have tested this item ✅ successfully on 0549e4b

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44491.}

[chmst]

I am not in favour of this solution.
The alt text now repeats the name of the contact. This is no help or information

It wold be better to use an empty alt-text or to use our usual solution where the user can decide if the image is only decorative or add useful information for example "smiling lady with greay hair" or "access to my office has two steps" instead of repeating the contacts name.

[brianteeman]

@chmst I agree with you - but that is a different issue which needs to be addressed for all the images - this PR is ONLY for the missing text on the email image.

The alt text on the contact image is nothing to do wioth this PR and there is a seperate issue/pull request elsewhere for that.

Comments about using alt titles and possibly invalid DT elements when using images instead of icons is beyond the scope of this PR. That is a bigger issue for the a11y team to consider

Please please dont make the mistake I made before and prevent this fix because you are looking at a completely different issue.

[chmst]

@brianteeman I know the complexity, this is why I did not touch this issue ;) The contact component needs a lot of work.
I think that alt="" would be better, but I will not block your solution.

[brianteeman]

PLEASE - look at this PR and what it does. It fixes a simple bug. It does nothing more but thanks to your completely off topic and irrelevant post talking about something com0pletely different its no wonder its been left for so long. Wish I hadnt bothered.

[QuyTon]

I have tested this item ✅ successfully on 1fb0196

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44491.}

[QuyTon]

Note: In 5.3, alt text is removed: #44523

[QuyTon]

RTC

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44491.}

[Hackwar]

Thank you for this fix!