WordPress.org

Make WordPress Core

Ticket #10910: 10910.005.diff

File 10910.005.diff, 8.3 KB (added by beaulebens, 4 years ago)

The attached patch takes a swing at fixing the semanticness of the comment_form output, based largely on Otto42's patch. Also includes all the CSS modifications required for 2010 to look the same as it does currently with the new defaults form output.

  • wp-includes/comment-template.php

     
    14651465 
    14661466        $req = get_option( 'require_name_email' ); 
    14671467        $aria_req = ( $req ? " aria-required='true'" : '' ); 
    1468         $defaults = array( 'fields' => apply_filters( 'comment_form_default_fields', array( 'author' => '<div id="form-section-author" class="form-section"> 
    1469                                                                                                                                                                                                                 <div class="form-label"><label for="author">' . __( 'Name', 'twentyten' ) . '</label> ' . ( $req ? '<span class="required">*</span>' : '' ) . '</div> 
    1470                                                                                                                                                                                                                 <div class="form-input"><input id="author" name="author" type="text" value="' . esc_attr( $commenter['comment_author'] ) . '" size="30" tabindex="1"' . $aria_req . ' /></div> 
    1471                                                                                                                                                                                                         </div><!-- #form-section-author .form-section -->', 
    1472                                                                                                                                                                                         'email' => '<div id="form-section-email" class="form-section"> 
    1473                                                                                                                                                                                                                         <div class="form-label"><label for="email">' . __( 'Email', 'twentyten' ) . '</label> ' . ( $req ? '<span class="required">*</span>' : '' ) . '</div> 
    1474                                                                                                                                                                                                                         <div class="form-input"><input id="email" name="email" type="text" value="' . esc_attr(  $commenter['comment_author_email'] ) . '" size="30" tabindex="2"' . $aria_req . ' /></div> 
    1475                                                                                                                                                                                                                 </div><!-- #form-section-email .form-section -->',  
    1476                                                                                                                                                                                         'url' => '<div id="form-section-url" class="form-section"> 
    1477                                                                                                                                                                                                                         <div class="form-label"><label for="url">' . __( 'Website', 'twentyten' ) . '</label></div> 
    1478                                                                                                                                                                                                                         <div class="form-input"><input id="url" name="url" type="text" value="' . esc_attr( $commenter['comment_author_url'] ) . '" size="30" tabindex="3" /></div> 
    1479                                                                                                                                                                                                                 </div><!-- #form-section-url .form-section -->' ) ), 
    1480                                                 'comment_field' => '<div id="form-section-comment" class="form-section"> 
    1481                                                                                                 <div class="form-label"><label for="comment">' . __( 'Comment', 'twentyten' ) . '</label></div> 
    1482                                                                                                 <div class="form-textarea"><textarea id="comment" name="comment" cols="45" rows="8" tabindex="4" aria-required="true"></textarea></div> 
    1483                                                                                         </div><!-- #form-section-comment .form-section -->', 
    1484                                                 'must_log_in' => '<p>' .  sprintf( __( 'You must be <a href="%s">logged in</a> to post a comment.' ), wp_login_url( apply_filters( 'the_permalink', get_permalink( $post_id ) ) ) ) . '</p>', 
    1485                                                 'logged_in_as' => '<p>' . sprintf( __( 'Logged in as <a href="%s">%s</a>. <a href="%s" title="Log out of this account">Log out?</a></p>' ), admin_url( 'profile.php' ), $user_identity, wp_logout_url( apply_filters( 'the_permalink', get_permalink( $post_id ) ) ) ), 
    1486                                                 'comment_notes_before' => '<p id="comment-notes">' . __( 'Your email is <em>never</em> published nor shared.' ) . ( $req ? __( ' Required fields are marked <span class="required">*</span>' ) : '' ) . '</p>',  
    1487                                                 'comment_notes_after' => '<div id="form-allowed-tags" class="form-section"> 
    1488                                                                                                         <p><span>' . __( 'You may use these <abbr title="HyperText Markup Language">HTML</abbr> tags and attributes:' ) . '</span> <code>' . allowed_tags() . '</code></p> 
    1489                                                                                                 </div>', 
     1468        $defaults = array( 'fields' => apply_filters( 'comment_form_default_fields', array( 'author' => '<p class="comment-form-author">' . 
     1469                                                                                                                                                                                                        '<label for="author">' . __( 'Name' ) . '</label> ' . 
     1470                                                                                                                                                                                                        ( $req ? '<span class="required">*</span>' : '' ) . 
     1471                                                                                                                                                                                                        '<input id="author" name="author" type="text" value="' . esc_attr( $commenter['comment_author'] ) . '" size="30" tabindex="1"' . $aria_req . ' />' .  
     1472                                                                                                                                                                                                        '</p><!-- #form-section-author .form-section -->', 
     1473                                                                                                                                                                                        'email' => '<p class="comment-form-email">' . 
     1474                                                                                                                                                                                                                '<label for="email">' . __( 'Email' ) . '</label> ' . 
     1475                                                                                                                                                                                                                ( $req ? '<span class="required">*</span>' : '' ) . 
     1476                                                                                                                                                                                                                '<input id="email" name="email" type="text" value="' . esc_attr(  $commenter['comment_author_email'] ) . '" size="30" tabindex="2"' . $aria_req . ' />' . 
     1477                                                                                                                                                                                                                '</p><!-- #form-section-email .form-section -->',  
     1478                                                                                                                                                                                        'url' => '<p class="comment-form-url">' . 
     1479                                                                                                                                                                                                                '<label for="url">' . __( 'Website' ) . '</label>' . 
     1480                                                                                                                                                                                                                '<input id="url" name="url" type="text" value="' . esc_attr( $commenter['comment_author_url'] ) . '" size="30" tabindex="3" />' . 
     1481                                                                                                                                                                                                                '</p><!-- #form-section-url .form-section -->' ) ), 
     1482                                                'comment_field' => '<p class="comment-form-comment">' . 
     1483                                                                                                '<label for="comment">' . __( 'Comment' ) . '</label>' . 
     1484                                                                                                '<textarea id="comment" name="comment" cols="45" rows="8" tabindex="4" aria-required="true"></textarea>' . 
     1485                                                                                        '</p><!-- #form-section-comment .form-section -->', 
     1486                                                'must_log_in' => '<p class="must-log-in">' .  sprintf( __( 'You must be <a href="%s">logged in</a> to post a comment.' ), wp_login_url( apply_filters( 'the_permalink', get_permalink( $post_id ) ) ) ) . '</p>', 
     1487                                                'logged_in_as' => '<p class="logged-in-as">' . sprintf( __( 'Logged in as <a href="%s">%s</a>. <a href="%s" title="Log out of this account">Log out?</a></p>' ), admin_url( 'profile.php' ), $user_identity, wp_logout_url( apply_filters( 'the_permalink', get_permalink( $post_id ) ) ) ), 
     1488                                                'comment_notes_before' => '<p class="comment-notes">' . __( 'Your email is <em>never</em> published nor shared.' ) . ( $req ? __( ' Required fields are marked <span class="required">*</span>' ) : '' ) . '</p>',  
     1489                                                'comment_notes_after' => '<dl class="form-allowed-tags"><dt>' . __( 'You may use these <abbr title="HyperText Markup Language">HTML</abbr> tags and attributes:' ) . '</dt> <dd><code>' . allowed_tags() . '</code></dd></dl>', 
    14901490                                                'id_form' => 'commentform', 
    14911491                                                'id_submit' => 'submit', 
    14921492                                                'title_reply' => __( 'Leave a Reply' ), 
    1493                                                 'title_reply_to' => __( 'Leave a Reply to %s'), 
     1493                                                'title_reply_to' => __( 'Leave a Reply to %s' ), 
    14941494                                                'cancel_reply_link' => __( 'Cancel reply' ), 
    14951495                                                'label_submit' => __( 'Post Comment' ), 
    14961496                                ); 
     
    15221522                                                <?php endif; ?> 
    15231523                                                <?php echo apply_filters( 'comment_form_field_comment', $args['comment_field'] ); ?> 
    15241524                                                <?php echo $args['comment_notes_after']; ?> 
    1525                                                 <div class="form-submit"> 
     1525                                                <p class="form-submit"> 
    15261526                                                        <input name="submit" type="submit" id="<?php echo esc_attr( $args['id_submit'] ); ?>" tabindex="<?php echo ( count( $args['fields'] ) + 2 ); ?>" value="<?php echo esc_attr( $args['label_submit'] ); ?>" /> 
    15271527                                                        <?php comment_id_fields(); ?> 
    1528                                                 </div> 
     1528                                                </p> 
    15291529                                                <?php do_action( 'comment_form', $post_id ); ?> 
    15301530                                        </form> 
    15311531                                <?php endif; ?> 
  • wp-content/themes/twentyten/style.css

     
    104104h3#reply-title, 
    105105#access .menu, 
    106106#cancel-comment-reply-link, 
    107 #form-allowed-tags, 
     107.form-allowed-tags, 
    108108#site-info, 
    109109#site-title, 
    110110#wp-calendar, 
     
    117117.entry-meta, 
    118118.entry-title, 
    119119.entry-utility, 
    120 .form-label, 
     120#respond label, 
    121121.navigation, 
    122122.page-title, 
    123123.pingback p, 
     
    962962        overflow: hidden; 
    963963        position: relative; 
    964964} 
     965#respond p { 
     966        margin: 0; 
     967} 
     968#respond .comment-notes { 
     969        margin-bottom: 1em; 
     970} 
    965971.children #respond { 
    966972        margin: 0 48px 0 0; 
    967973} 
     
    983989        color:#FF4B33; 
    984990        font-weight:bold; 
    985991} 
    986 .form-section label { 
     992#respond label { 
    987993        color: #888; 
    988994        font-size: 12px; 
    989995} 
    990 .form-section input { 
     996#respond input { 
    991997        margin:0 0 9px 0; 
    992998        width:98%; 
    993999} 
    994 .form-section textarea { 
     1000#respond textarea { 
    9951001        width:99%; 
    9961002} 
    997 #form-allowed-tags { 
    998         color:#888; 
     1003#respond dt, 
     1004#respond dd { 
     1005        color: #888; 
    9991006        font-size: 12px; 
    10001007        line-height: 18px; 
    10011008} 
    1002 #form-allowed-tags p { 
    1003         margin:0; 
     1009#respond dd code { 
     1010        font-size: 11px; 
    10041011} 
    1005 #form-allowed-tags span { 
     1012#respond .form-submit { 
     1013        margin: 0 0 24px; 
    10061014} 
    1007 #form-allowed-tags code { 
    1008         font-size: 11px; 
     1015#respond .form-submit input { 
     1016        width: auto; 
    10091017} 
    1010 .form-submit { 
    1011         margin: 12px 0 24px 0; 
     1018#respond dl { 
     1019        margin: 0 0 10px; 
    10121020} 
     1021#respond dt { 
     1022         display: inline; 
     1023        font-weight: normal; 
     1024} 
     1025#respond dd { 
     1026        display: inline; 
     1027} 
    10131028 
    10141029 
    10151030