Changeset 22123

Timestamp:

10/05/2012 04:34:32 PM (13 years ago)

Author:

ryan

Message:

Put required field indicators within labels. Update styling of bundled themes to accommodate. Props MikeHansenMe. fixes #21897

Location:

trunk

Files:

• wp-content/themes/twentyeleven/style.css (modified) (1 diff)
• wp-content/themes/twentytwelve/style.css (modified) (1 diff)
• wp-includes/comment-template.php (modified) (1 diff)

trunk/wp-content/themes/twentyeleven/style.css

@@ -2124 +2124 @@
     left: 75%;
     position: absolute;
-    top: 45px;
     z-index: 1;
 }

trunk/wp-content/themes/twentytwelve/style.css

@@ -1238 +1238 @@
     color: red;
 }
-label ~ span.required {
-    display: block;
-    float: left;
-    margin: -18px 0 0 -16px;
-    margin: -1.285714286rem 0 0 -1.142857143rem;
-}
 
 

trunk/wp-includes/comment-template.php

@@ -1522 +1522 @@
     $aria_req = ( $req ? " aria-required='true'" : '' );
     $fields =  array(
-        'author' => '<p class="comment-form-author">' . '<label for="author">' . __( 'Name' ) . '</label> ' . ( $req ? '<span class="required">*</span>' : '' ) .
+        'author' => '<p class="comment-form-author">' . '<label for="author">' . __( 'Name' ) . ( $req ? ' <span class="required">*</span>' : '' ) . '</label> ' .
                     '<input id="author" name="author" type="text" value="' . esc_attr( $commenter['comment_author'] ) . '" size="30"' . $aria_req . ' /></p>',
-        'email'  => '<p class="comment-form-email"><label for="email">' . __( 'Email' ) . '</label> ' . ( $req ? '<span class="required">*</span>' : '' ) .
+        'email'  => '<p class="comment-form-email"><label for="email">' . __( 'Email' ) . ( $req ? ' <span class="required">*</span>' : '' ) . '</label> ' .
                     '<input id="email" name="email" type="text" value="' . esc_attr(  $commenter['comment_author_email'] ) . '" size="30"' . $aria_req . ' /></p>',
         'url'    => '<p class="comment-form-url"><label for="url">' . __( 'Website' ) . '</label>' .