Skip to:
Content

bbPress.org

Changeset 5948


Ignore:
Timestamp:
10/28/2015 01:17:06 AM (5 years ago)
Author:
netweb
Message:

Accessibility: Include input labels for field titles in bbPress settings

This changeset aligns bbPress input label fields with current WordPress standard for improved accessibility

Props ankit-k-gupta, thebrandonallen
Fixes #2866

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/includes/admin/settings.php

    r5908 r5948  
    9292                'callback'          => 'bbp_admin_setting_callback_editlock',
    9393                'sanitize_callback' => 'intval',
    94                 'args'              => array()
     94                'args'              => array( 'label_for' => '_bbp_edit_lock' )
    9595            ),
    9696
     
    100100                'callback'          => 'bbp_admin_setting_callback_throttle',
    101101                'sanitize_callback' => 'intval',
    102                 'args'              => array()
     102                'args'              => array( 'label_for' => '_bbp_throttle_time' )
    103103            ),
    104104
     
    231231                'callback'          => 'bbp_admin_setting_callback_topics_per_page',
    232232                'sanitize_callback' => 'intval',
    233                 'args'              => array()
     233                'args'              => array( 'label_for' => '_bbp_topics_per_page' )
    234234            ),
    235235
     
    239239                'callback'          => 'bbp_admin_setting_callback_replies_per_page',
    240240                'sanitize_callback' => 'intval',
    241                 'args'              => array()
     241                'args'              => array( 'label_for' => '_bbp_replies_per_page' )
    242242            )
    243243        ),
     
    252252                'callback'          => 'bbp_admin_setting_callback_topics_per_rss_page',
    253253                'sanitize_callback' => 'intval',
    254                 'args'              => array()
     254                'args'              => array( 'label_for' => '_bbp_topics_per_rss_page' )
    255255            ),
    256256
     
    260260                'callback'          => 'bbp_admin_setting_callback_replies_per_rss_page',
    261261                'sanitize_callback' => 'intval',
    262                 'args'              => array()
     262                'args'              => array( 'label_for' => '_bbp_replies_per_rss_page' )
    263263            )
    264264        ),
     
    273273                'callback'          => 'bbp_admin_setting_callback_root_slug',
    274274                'sanitize_callback' => 'bbp_sanitize_slug',
    275                 'args'              => array()
     275                'args'              => array( 'label_for' => '_bbp_root_slug' )
    276276            ),
    277277
     
    289289                'callback'          => 'bbp_admin_setting_callback_show_on_root',
    290290                'sanitize_callback' => 'sanitize_text_field',
    291                 'args'              => array()
     291                'args'              => array( 'label_for'=>'_bbp_show_on_root' )
    292292            ),
    293293        ),
     
    302302                'callback'          => 'bbp_admin_setting_callback_forum_slug',
    303303                'sanitize_callback' => 'bbp_sanitize_slug',
    304                 'args'              => array()
     304                'args'              => array( 'label_for'=>'_bbp_forum_slug' )
    305305            ),
    306306
     
    310310                'callback'          => 'bbp_admin_setting_callback_topic_slug',
    311311                'sanitize_callback' => 'bbp_sanitize_slug',
    312                 'args'              => array()
     312                'args'              => array( 'label_for'=>'_bbp_topic_slug' )
    313313            ),
    314314
     
    318318                'callback'          => 'bbp_admin_setting_callback_topic_tag_slug',
    319319                'sanitize_callback' => 'bbp_sanitize_slug',
    320                 'args'              => array()
     320                'args'              => array( 'label_for'=>'_bbp_topic_tag_slug' )
    321321            ),
    322322
     
    326326                'callback'          => 'bbp_admin_setting_callback_view_slug',
    327327                'sanitize_callback' => 'bbp_sanitize_slug',
    328                 'args'              => array()
     328                'args'              => array( 'label_for'=>'_bbp_view_slug' )
    329329            ),
    330330
     
    334334                'callback'          => 'bbp_admin_setting_callback_reply_slug',
    335335                'sanitize_callback' => 'bbp_sanitize_slug',
    336                 'args'              => array()
     336                'args'              => array( 'label_for'=>'_bbp_reply_slug' )
    337337            ),
    338338
     
    342342                'callback'          => 'bbp_admin_setting_callback_search_slug',
    343343                'sanitize_callback' => 'bbp_sanitize_slug',
    344                 'args'              => array()
     344                'args'              => array( 'label_for'=>'_bbp_search_slug' )
    345345            )
    346346        ),
     
    355355                'callback'          => 'bbp_admin_setting_callback_user_slug',
    356356                'sanitize_callback' => 'bbp_sanitize_slug',
    357                 'args'              => array()
     357                'args'              => array( 'label_for'=>'_bbp_user_slug' )
    358358            ),
    359359
     
    363363                'callback'          => 'bbp_admin_setting_callback_topic_archive_slug',
    364364                'sanitize_callback' => 'bbp_sanitize_slug',
    365                 'args'              => array()
     365                'args'              => array( 'label_for'=>'_bbp_topic_archive_slug' )
    366366            ),
    367367
     
    371371                'callback'          => 'bbp_admin_setting_callback_reply_archive_slug',
    372372                'sanitize_callback' => 'bbp_sanitize_slug',
    373                 'args'              => array()
     373                'args'              => array( 'label_for'=>'_bbp_reply_archive_slug' )
    374374            ),
    375375
     
    379379                'callback'          => 'bbp_admin_setting_callback_user_favs_slug',
    380380                'sanitize_callback' => 'bbp_sanitize_slug',
    381                 'args'              => array()
     381                'args'              => array( 'label_for'=>'_bbp_user_favs_slug' )
    382382            ),
    383383
     
    387387                'callback'          => 'bbp_admin_setting_callback_user_subs_slug',
    388388                'sanitize_callback' => 'bbp_sanitize_slug',
    389                 'args'              => array()
     389                'args'              => array( 'label_for'=>'_bbp_user_subs_slug' )
    390390            )
    391391        ),
     
    408408                'callback'          => 'bbp_admin_setting_callback_group_forums_root_id',
    409409                'sanitize_callback' => 'intval',
    410                 'args'              => array()
     410                'args'              => array( 'label_for'=>'_bbp_group_forums_root_id' )
    411411            )
    412412        ),
     
    476476
    477477    <input name="_bbp_edit_lock" id="_bbp_edit_lock" type="number" min="0" step="1" value="<?php bbp_form_option( '_bbp_edit_lock', '5' ); ?>" class="small-text"<?php bbp_maybe_admin_setting_disabled( '_bbp_edit_lock' ); ?> />
    478     <label for="_bbp_edit_lock"><?php esc_html_e( 'minutes', 'bbpress' ); ?></label>
     478    <?php esc_html_e( 'minutes', 'bbpress' ); ?>
    479479
    480480<?php
     
    492492
    493493    <input name="_bbp_throttle_time" id="_bbp_throttle_time" type="number" min="0" step="1" value="<?php bbp_form_option( '_bbp_throttle_time', '10' ); ?>" class="small-text"<?php bbp_maybe_admin_setting_disabled( '_bbp_throttle_time' ); ?> />
    494     <label for="_bbp_throttle_time"><?php esc_html_e( 'seconds', 'bbpress' ); ?></label>
     494    <?php esc_html_e( 'seconds', 'bbpress' ); ?>
    495495
    496496<?php
     
    804804
    805805    <input name="_bbp_topics_per_page" id="_bbp_topics_per_page" type="number" min="1" step="1" value="<?php bbp_form_option( '_bbp_topics_per_page', '15' ); ?>" class="small-text"<?php bbp_maybe_admin_setting_disabled( '_bbp_topics_per_page' ); ?> />
    806     <label for="_bbp_topics_per_page"><?php esc_html_e( 'per page', 'bbpress' ); ?></label>
     806    <?php esc_html_e( 'per page', 'bbpress' ); ?>
    807807
    808808<?php
     
    820820
    821821    <input name="_bbp_replies_per_page" id="_bbp_replies_per_page" type="number" min="1" step="1" value="<?php bbp_form_option( '_bbp_replies_per_page', '15' ); ?>" class="small-text"<?php bbp_maybe_admin_setting_disabled( '_bbp_replies_per_page' ); ?> />
    822     <label for="_bbp_replies_per_page"><?php esc_html_e( 'per page', 'bbpress' ); ?></label>
     822    <?php esc_html_e( 'per page', 'bbpress' ); ?>
    823823
    824824<?php
     
    851851
    852852    <input name="_bbp_topics_per_rss_page" id="_bbp_topics_per_rss_page" type="number" min="1" step="1" value="<?php bbp_form_option( '_bbp_topics_per_rss_page', '25' ); ?>" class="small-text"<?php bbp_maybe_admin_setting_disabled( '_bbp_topics_per_rss_page' ); ?> />
    853     <label for="_bbp_topics_per_rss_page"><?php esc_html_e( 'per page', 'bbpress' ); ?></label>
     853    <?php esc_html_e( 'per page', 'bbpress' ); ?>
    854854
    855855<?php
     
    867867
    868868    <input name="_bbp_replies_per_rss_page" id="_bbp_replies_per_rss_page" type="number" min="1" step="1" value="<?php bbp_form_option( '_bbp_replies_per_rss_page', '25' ); ?>" class="small-text"<?php bbp_maybe_admin_setting_disabled( '_bbp_replies_per_rss_page' ); ?> />
    869     <label for="_bbp_replies_per_rss_page"><?php esc_html_e( 'per page', 'bbpress' ); ?></label>
     869    <?php esc_html_e( 'per page', 'bbpress' ); ?>
    870870
    871871<?php
     
    12281228    ) ); ?>
    12291229
    1230     <label for="_bbp_group_forums_root_id"><?php esc_html_e( 'is the parent for all group forums', 'bbpress' ); ?></label>
     1230    <?php esc_html_e( 'is the parent for all group forums', 'bbpress' ); ?>
    12311231    <p class="description"><?php esc_html_e( 'Using the Forum Root is not recommended. Changing this does not move existing forums.', 'bbpress' ); ?></p>
    12321232
Note: See TracChangeset for help on using the changeset viewer.