• Amber Hinds

    (@alh0319)


    Hi,

    Thanks for the handy plugin! Can you please make this fix to improve that accessibility of your forms?

    The list field is missing a fieldset and legend, and has an orphaned label that is not associated with any field. This is the current code:

    <div id="ic34d" class="gjs-row">
    <div class="gjs-cell">
    <label id="" class="es-field-label">What type of posts would you like to get notified about? (Required)</label>
    <div id="undefined" class="es-list">
    <div><label><input type="checkbox" class="pl-6 form-checkbox " name="esfpx_lists[]" value="ac95f3d9ac71"><span>WordPress</span></label></div>
    <div><label><input type="checkbox" class="pl-6 form-checkbox " name="esfpx_lists[]" value="c4dfe0f151fc"><span>Business</span></label></div>
    <div><label><input type="checkbox" class="pl-6 form-checkbox " name="esfpx_lists[]" value="c1e876a12681"><span>Life/Personal</span></label></div>
    </div>
    </div>
    </div>

    This is what it should be:

    <fieldset id="ic34d" class="gjs-row">
    <div class="gjs-cell">
    <legend id="" class="es-field-label">What type of posts would you like to get notified about? (Required)</legend>
    <div id="undefined" class="es-list">
    <div><label><input type="checkbox" class="pl-6 form-checkbox " name="esfpx_lists[]" value="ac95f3d9ac71"><span>WordPress</span></label></div>
    <div><label><input type="checkbox" class="pl-6 form-checkbox " name="esfpx_lists[]" value="c4dfe0f151fc"><span>Business</span></label></div>
    <div><label><input type="checkbox" class="pl-6 form-checkbox " name="esfpx_lists[]" value="c1e876a12681"><span>Life/Personal</span></label></div>
    </div>
    </div>
    </fieldset>

    The fieldset is necessary for screen reader users to understand how fields are grouped and is required for conformance with Web Content Accessibility Guidelines. Fixing this would be a big improvement for all websites using your plugin. Thank you!

Viewing 1 replies (of 1 total)
Viewing 1 replies (of 1 total)
  • You must be logged in to reply to this topic.