• Resolved morriscountynj

    (@morriscountynj)


    I use an automated accessibility checker called Siteimprove to keep tabs on issues with my websites, and I noticed a couple of biggies with this plugin, unfortunately:

    1) Input field has no description
    The checkbox next to “yes, i accept your privacy policy” isn’t labeled, or marked by aria-labelledby. report from siteimprove

    2) Label is not connected to a form control
    The label on “yes, i accept your privacy policy” doesn’t reference an existing id on the checkbox. report from siteimprove

    3) Missing button in form
    The form has a link, not an actual semantic <button>, for submitting, which isn’t accessible. report from siteimprove

    I was wondering if you could take a look at these and make fixes to the plugin? I’d made fixes myself, but then they get overwritten every time I update. 🙂 Thank you

    The page I need help with: [log in to see the link]

Viewing 3 replies - 1 through 3 (of 3 total)
  • Plugin Author SpeakOut! Steve

    (@123host)

    Hi, thanks for the feedback. If you would like to make the changes and send me the code, as long as it is suitable I am happy to include it in the code.

    As for the button, yes it is not an actual form button. A few years ago I adopted this abandoned plugin to keep it alive. That is how the original author had it functioning. I have no objection to it becoming a real button as long as there are no knock-on effects.

    I look forward to your contribution.

    Plugin Author SpeakOut! Steve

    (@123host)

    Nothing heard :o(

    Thread Starter morriscountynj

    (@morriscountynj)

    My apologies for not getting back!

    Try this out, working on my end:

    includes/emailpetition.php

    line 272 is originally:
    <a name="' . $petition->id . '" class="dk-speakout-submit"><span>' . stripslashes( esc_html( $options['button_text'] ) ) . '</span></a>

    CHANGE IT to:
    <button name="' . $petition->id . '" class="dk-speakout-submit">' . stripslashes( esc_html( $options['button_text'] ) ) . '</button>

Viewing 3 replies - 1 through 3 (of 3 total)

The topic ‘Accessibility issues’ is closed to new replies.