Empty img alt="" attribute automatically removed on entry save.

[jlcharette]

Description

Empty alt="" attributes on images get removed.

I found in https://github.com/craftcms/redactor/blob/v2/src/assets/field/src/js/RedactorOverrides.js
the following snippet:

    // remove image attributes
    var $wrapper = this.utils.buildWrapper(html);
    var imageattrs = ['alt', 'title', 'src', 'class', 'width', 'height', 'srcset', 'style'];
    $wrapper.find('img').each(function(node) {
        if (node.attributes.length > 0) {
            var attrs = node.attributes;
            for (var i = attrs.length - 1; i >= 0; i--) {
                if (attrs[i].name.search(/^data\-/) === -1 && imageattrs.indexOf(attrs[i].name) === -1) {
                    node.removeAttribute(attrs[i].name);
                }
            }
        }
    });

Which seems to be the cause of my issue.

Is there a way to override this so the empty alt="" attr is allowed? I absolutely need to follow this guideline:
https://www.w3.org/WAI/WCAG21/Techniques/html/H67.html

Steps to reproduce

1. Add an image
2. Insert alt="" via the code view
3. Save entry
4. The alt tag has been removed

Additional info

• Craft version: 3.4.30
• PHP version: 7.4.8
• Database driver & version: MySQL 8.0.21
• Plugins & versions: Redactor 2.6.1

[andris-sevcenko]

Just pushed a fix for this. I'll see if I can get a Redactor release out next Tuesday!

[bryanluu]

How would I go about removing the alt="" attribute entirely? I'm hoping to make Assets uploaded directly into a Redactor field use the Asset's alt-text. Currently when I upload the asset into a Redactor field, it uses the "Title" field from Redactor's uploader to act as the asset's alt-text.

---

(Update)
I was able to find an answer on the Craft Stack exchange:

• https://craftcms.stackexchange.com/questions/4763/adding-alt-tag-to-an-image
• https://craftcms.stackexchange.com/questions/4559/is-there-a-way-integrate-redactor-with-your-assets-fields

It seems like that capability isn't implemented in Craft yet, to have Redactor fields use the asset's alt-text. Perhaps an issue/feature suggestion is to be able to configure the Upload fields in a Redactor field? So changing the "Title" text to be "Title (used as alternative text)" or something.