Changeset 21584

Timestamp:

08/22/2012 05:40:17 PM (12 years ago)

Author:

azaozz

Message:

Don't hide the label for the Title on the Edit Post screen, use 'screen-reader-text' instead, props SergeyBiryukov, fixes #21458

Location:

trunk/wp-admin

Files:

• css/wp-admin.dev.css (modified) (1 diff)
• edit-form-advanced.php (modified) (1 diff)
• js/post.dev.js (modified) (2 diffs)

trunk/wp-admin/css/wp-admin.dev.css

@@ -2828 +2828 @@
 
 #wp-fullscreen-title-prompt-text {
-    left: 0;
     padding: 11px;
 }

trunk/wp-admin/edit-form-advanced.php

@@ -283 +283 @@
 <div id="titlediv">
 <div id="titlewrap">
-    <label class="hide-if-no-js" style="visibility:hidden" id="title-prompt-text" for="title"><?php echo apply_filters( 'enter_title_here', __( 'Enter title here' ), $post ); ?></label>
+    <label class="screen-reader-text" id="title-prompt-text" for="title"><?php echo apply_filters( 'enter_title_here', __( 'Enter title here' ), $post ); ?></label>
     <input type="text" name="post_title" size="30" value="<?php echo esc_attr( htmlspecialchars( $post->post_title ) ); ?>" id="title" autocomplete="off" />
 </div>

trunk/wp-admin/js/post.dev.js

@@ -642 +642 @@
 
         if ( title.val() == '' )
-            titleprompt.css('visibility', '');
+            titleprompt.removeClass('screen-reader-text');
 
         titleprompt.click(function(){
-            $(this).css('visibility', 'hidden');
+            $(this).addClass('screen-reader-text');
             title.focus();
         });
@@ -651 +651 @@
         title.blur(function(){
             if ( this.value == '' )
-                titleprompt.css('visibility', '');
+                titleprompt.removeClass('screen-reader-text');
         }).focus(function(){
-            titleprompt.css('visibility', 'hidden');
+            titleprompt.addClass('screen-reader-text');
         }).keydown(function(e){
-            titleprompt.css('visibility', 'hidden');
+            titleprompt.addClass('screen-reader-text');
             $(this).unbind(e);
         });