WCAG validation

  • I’ve spotted some minor problems in the code generated by WP Form Manager which means that it doesn’t satisfy WCAG Priority 1 (so it technically illegal in most developed countries).

    Here are the required fixes:

    types/recaptcha/recaptchalib.php (line 122)

    return ‘<script type=”text/javascript” src=”‘. $server . ‘/challenge?k=’ . $pubkey . $errorpart . ‘”></script>
    <noscript>
    <iframe src=”‘. $server . ‘/noscript?k=’ . $pubkey . $errorpart . ‘” height=”300″ width=”500″ frameborder=”0″>Recaptcha</iframe>
    <textarea name=”recaptcha_challenge_field” rows=”3″ cols=”40″></textarea>
    <input type=”hidden” name=”recaptcha_response_field” value=”manual_challenge”/>
    </noscript>’;

    templates/fm-form-default.php(line 88)
    <label style=”display:block;width:<?php echo $labelwidth;?>px;” for=”<?php echo fm_form_the_ID(); ?>”><?php echo fm_form_the_label(); ?>
    (line 93)
    <td style=”width:<?php echo $labelWidth; ?>px”><label for=”<?php echo fm_form_the_ID(); ?>”><?php echo fm_form_the_label(); ?><?php if(fm_form_is_required()) echo ” *“; ?></label></td>

    types/list.php(lines 81/91)
    ‘separator’ => ‘
    ‘,

    https://www.ads-software.com/extend/plugins/wordpress-form-manager/

Viewing 7 replies - 1 through 7 (of 7 total)
  • Plugin Author hoffcamp

    (@hoffcamp)

    Let me make sure I understand the changes you want:

    1) Add ‘Recaptcha’ between the iframe open/close tags
    2) Add ‘for’ attributes on the label tags in the default form template
    3) Change the list separator for checkbox/radio button lists from a br tag to a newline character

    I made changes 1 and 2 to my code, I assume these are about accessbility?

    For number 3, this would make checkbox/radio button lists render horizontally instead of vertically. I can’t make this change without breaking everybody’s forms, so the W3C is going to have to deal with my violation of WCAG Priority 1 on this issue.

    Thread Starter OpenGlobal

    (@openglobal)

    Whoops! Change (3) got interpreted as HTML, it should read:

    ‘separator’ => ‘
    ‘

    ie. self close it.

    Thread Starter OpenGlobal

    (@openglobal)

    For crying out loud! Why does it keep interpreting it as HTML??

    ‘separator’ => ‘
    ‘

    Thread Starter OpenGlobal

    (@openglobal)

    OK, let me try it this way, you need to change “br” to “br /”.

    Thread Starter OpenGlobal

    (@openglobal)

    'separator' => '<br />'

    Plugin Author hoffcamp

    (@hoffcamp)

    Ohhh okay. I’ll make that change as well. Thanks for pointing this stuff out.

    Thread Starter OpenGlobal

    (@openglobal)

    I’ve just discovered something else:

    formelements/formelements.php(line 93):

    $arr[] = “<label><input type=\”checkbox\” “.fe_getAttributeString($elementDef[‘attributes’]).” id=\””.htmlspecialchars($k).”\” name=\””.htmlspecialchars($k).”\” “.($vals[$k]?’checked’:”).”/>  “.htmlspecialchars($v).”</label>”;

Viewing 7 replies - 1 through 7 (of 7 total)
  • The topic ‘WCAG validation’ is closed to new replies.