Changeset 30842

Timestamp:

12/13/2014 10:07:58 PM (7 years ago)

Author:

azaozz

Message:

Do not autofocus text fields on page load on mobile devices. This is currently broken in many mobile browsers:

• iOS Safari opens the keyboard and auto-scrolls on the first tap anywhere on the screen triggering the click at an unexpected place. That makes it impossible to follow links or press buttons.
• Chrome on iOS opens the keyboard on load and may scroll the focused field off screen.
• The Android 4.4 browser only highlights the field, the user has to tap it to open the keyboard and type.

See #30703.

Location:

trunk/src/wp-admin

Files:

• edit-form-advanced.php (modified) (1 diff)
• edit-form-comment.php (modified) (1 diff)
• edit-tag-form.php (modified) (1 diff)
• edit-tags.php (modified) (1 diff)

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

@@ -648 +648 @@
 ?>
 
-<?php if ( post_type_supports( $post_type, 'title' ) && '' === $post->post_title ) : ?>
+<?php if ( ! wp_is_mobile() && post_type_supports( $post_type, 'title' ) && '' === $post->post_title ) : ?>
 <script type="text/javascript">
 try{document.post.title.focus();}catch(e){}

trunk/src/wp-admin/edit-form-comment.php

@@ -156 +156 @@
 </form>
 
+<?php if ( ! wp_is_mobile() ) : ?>
 <script type="text/javascript">
 try{document.post.name.focus();}catch(e){}
 </script>
+<?php endif;

trunk/src/wp-admin/edit-tag-form.php

@@ -201 +201 @@
 </form>
 </div>
+
+<?php if ( ! wp_is_mobile() ) : ?>
 <script type="text/javascript">
 try{document.forms.edittag.name.focus();}catch(e){}
 </script>
+<?php endif;

trunk/src/wp-admin/edit-tags.php

@@ -584 +584 @@
 </div><!-- /col-container -->
 </div><!-- /wrap -->
+
+<?php if ( ! wp_is_mobile() ) : ?>
 <script type="text/javascript">
 try{document.forms.addtag['tag-name'].focus();}catch(e){}
 </script>
 <?php
+endif;
+
 $wp_list_table->inline_edit();