Changeset 10258

Timestamp:

12/27/2008 12:18:38 PM (14 years ago)

Author:

azaozz

Message:

Faster hide-if-no-js, if-js-closed, etc. needed when loading scripts in the footer.

Location:

trunk

Files:

• wp-admin/admin-header.php (modified) (1 diff)
• wp-admin/css/global.css (modified) (1 diff)
• wp-admin/css/no-js.css (added)
• wp-admin/includes/post.php (modified) (1 diff)
• wp-admin/js/comment.js (modified) (1 diff)
• wp-admin/js/common.js (modified) (1 diff)
• wp-admin/js/dashboard.js (modified) (2 diffs)
• wp-admin/js/edit-comments.js (modified) (1 diff)
• wp-admin/js/inline-edit-post.js (modified) (1 diff)
• wp-admin/js/inline-edit-tax.js (modified) (2 diffs)
• wp-admin/js/link.js (modified) (1 diff)
• wp-admin/js/page.js (modified) (1 diff)
• wp-admin/js/post.js (modified) (2 diffs)
• wp-admin/wp-admin.css (modified) (3 diffs)
• wp-includes/class.wp-styles.php (modified) (3 diffs)
• wp-includes/script-loader.php (modified) (11 diffs)

trunk/wp-admin/admin-header.php

@@ -73 +73 @@
 do_action('admin_head');
 
-if ( $is_iphone ) { ?>
+wp_print_styles('no-js');
+?>
+<script type="text/javascript">
+(function(){
+    var nojs = document.getElementById('no-js');
+    nojs.parentNode.removeChild(nojs);
+})();
+</script>
+
+<?php if ( $is_iphone ) { ?>
 <style type="text/css">.row-actions{visibility:visible;}</style>
 <?php } ?>

trunk/wp-admin/css/global.css

@@ -179 +179 @@
 .hidden,
 .closed .inside,
-.hide-if-no-js {
+.hide-if-js {
     display: none;
 }

trunk/wp-admin/includes/post.php

@@ -872 +872 @@
     if ( $closed = get_user_option('closedpostboxes_'.$page, 0, false ) ) {
         if ( !is_array( $closed ) ) return '';
-        return in_array( $id, $closed )? 'if-js-closed' : '';
+        return in_array( $id, $closed )? 'closed' : '';
     } else {
         return '';

trunk/wp-admin/js/comment.js

@@ -1 +1 @@
 jQuery(document).ready( function($) {
-    jQuery('.hide-if-no-js').show();
-    jQuery('.hide-if-js').hide();
 
     var stamp = $('#timestamp').html();

trunk/wp-admin/js/common.js

@@ -133 +133 @@
     // pulse
     $('.fade').animate( { backgroundColor: '#ffffe0' }, 300).animate( { backgroundColor: '#fffbcc' }, 300).animate( { backgroundColor: '#ffffe0' }, 300).animate( { backgroundColor: '#fffbcc' }, 300);
-
-    // show things that should be visible, hide what should be hidden
-    $('.hide-if-no-js').removeClass('hide-if-no-js');
-    $('.hide-if-js').hide();
 
     // Basic form validation

trunk/wp-admin/js/dashboard.js

@@ -1 +1 @@
 
 jQuery( function($) {
-
-// close postboxes that should be closed
-jQuery('.if-js-closed').removeClass('if-js-closed').addClass('closed');
 
 // These widgets are sometimes populated via ajax
@@ -38 +35 @@
                 $('#dashboard_recent_drafts ul').prepend( this );
             } ).end().remove();
-            $(this).find('.hide-if-no-js').removeClass('hide-if-no-js');
             tb_init('a.thickbox');
             quickPressLoad();

trunk/wp-admin/js/edit-comments.js

@@ -304 +304 @@
         $('#replyrow').after(c);
         this.o = id = '#comment-'+r.id;
-        $(id+' .hide-if-no-js').removeClass('hide-if-no-js');
         this.revert();
         this.addEvents($(id));

trunk/wp-admin/js/inline-edit-post.js

@@ -221 +221 @@
                             row.find('td.column-comments').hide();
 
-                        row.find('.hide-if-no-js').removeClass('hide-if-no-js');
                         inlineEditPost.addEvents(row);
                         row.fadeIn();

trunk/wp-admin/js/inline-edit-tax.js

@@ -37 +37 @@
         r.each(function() {
             $(this).find('a.editinline').click(function() { inlineEditTax.edit(this); return false; });
-            $(this).find('.hide-if-no-js').removeClass('hide-if-no-js');
         });
     },
@@ -117 +116 @@
                         row.hide();
 
-                        row.find('.hide-if-no-js').removeClass('hide-if-no-js');
                         inlineEditTax.addEvents(row);
                         row.fadeIn();

trunk/wp-admin/js/link.js

@@ -1 +1 @@
 jQuery(document).ready( function($) {
-    // close postboxes that should be closed
-    jQuery('.if-js-closed').removeClass('if-js-closed').addClass('closed');
 
     jQuery('#link_name').focus();

trunk/wp-admin/js/page.js

@@ -2 +2 @@
     postboxes.add_postbox_toggles('page');
     make_slugedit_clickable();
-
-    // close postboxes that should be closed
-    jQuery('.if-js-closed').removeClass('if-js-closed').addClass('closed');
 
     jQuery('#title').blur( function() { if ( (jQuery("#post_ID").val() > 0) || (jQuery("#title").val().length == 0) ) return; autosave(); } );

trunk/wp-admin/js/post.js

@@ -151 +151 @@
 jQuery(document).ready( function($) {
     tagCloud.init();
-
-    // close postboxes that should be closed
-    jQuery('.if-js-closed').removeClass('if-js-closed').addClass('closed');
 
     // postboxes
@@ -468 +465 @@
                     if ( 'object' == typeof r && r.responses[0] ) {
                         $('#the-comment-list').append( r.responses[0].data );
-                        $('#the-comment-list .hide-if-no-js').removeClass('hide-if-no-js');
 
                         theList = theExtraList = null;

trunk/wp-admin/wp-admin.css

@@ -671 +671 @@
 }
 
-.tablenav .tablenav-pages a, .tablenav-pages span.current  {
+.tablenav .tablenav-pages a,
+.tablenav-pages span.current  {
     text-decoration: none;
     border: none;
@@ -705 +706 @@
 }
 
-#update-nag, .plugin-update {
+#update-nag,
+.plugin-update {
     line-height: 29px;
     font-size: 12px;
@@ -715 +717 @@
 }
 
-#update-nag a, .plugin-update a {
+#update-nag a,
+.plugin-update a {
     font-size: 1.1em;
 }

trunk/wp-includes/class.wp-styles.php

@@ -40 +40 @@
 
         $href = $this->_css_href( $this->registered[$handle]->src, $ver, $handle );
+        $rel = isset($this->registered[$handle]->extra['alt']) && $this->registered[$handle]->extra['alt'] ? 'alternate stylesheet' : 'stylesheet';
+        $title = isset($this->registered[$handle]->extra['title']) ? "title='" . attribute_escape( $this->registered[$handle]->extra['title'] ) . "'" : '';
 
         $end_cond = '';
@@ -47 +49 @@
         }
 
-        echo apply_filters( 'style_loader_tag', "<link rel='stylesheet' href='$href' type='text/css' media='$media' />\n", $handle );
+        echo apply_filters( 'style_loader_tag', "<link rel='$rel' id='$handle' $title href='$href' type='text/css' media='$media' />\n", $handle );
         if ( 'rtl' === $this->text_direction && isset($this->registered[$handle]->extra['rtl']) && $this->registered[$handle]->extra['rtl'] ) {
             if ( is_bool( $this->registered[$handle]->extra['rtl'] ) )
@@ -54 +56 @@
                 $rtl_href = $this->_css_href( $this->registered[$handle]->extra['rtl'], $ver, "$handle-rtl" );
 
-            echo apply_filters( 'style_loader_tag', "<link rel='stylesheet' href='$rtl_href' type='text/css' media='$media' />\n", $handle );
+            echo apply_filters( 'style_loader_tag', "<link rel='$rel' id='$handle' $title href='$rtl_href' type='text/css' media='$media' />\n", $handle );
         }
 

trunk/wp-includes/script-loader.php

@@ -42 +42 @@
     $scripts->default_version = get_bloginfo( 'version' );
 
-    $scripts->add( 'common', '/wp-admin/js/common.js', array('jquery', 'hoverIntent'), '20081210' );
+    $scripts->add( 'common', '/wp-admin/js/common.js', array('jquery', 'hoverIntent'), '20081226' );
     $scripts->add( 'sack', '/wp-includes/js/tw-sack.js', false, '1.6.1' );
 
@@ -162 +162 @@
             'l10n_print_after' => 'try{convertEntities(pwsL10n);}catch(e){};'
         ) );
-        $scripts->add( 'admin-comments', '/wp-admin/js/edit-comments.js', array('wp-lists', 'jquery-ui-resizable', 'quicktags'), '20081211' );
+        $scripts->add( 'admin-comments', '/wp-admin/js/edit-comments.js', array('wp-lists', 'jquery-ui-resizable', 'quicktags'), '20081226' );
         $scripts->localize( 'admin-comments', 'adminCommentsL10n', array(
             'hotkeys_highlight_first' => isset($_GET['hotkeys_highlight_first']),
@@ -180 +180 @@
             'l10n_print_after' => 'try{convertEntities(slugL10n);}catch(e){};'
         ) );
-        $scripts->add( 'post', '/wp-admin/js/post.js', array('suggest', 'jquery-ui-tabs', 'wp-lists', 'postbox', 'slug'), '20081217' );
+        $scripts->add( 'post', '/wp-admin/js/post.js', array('suggest', 'jquery-ui-tabs', 'wp-lists', 'postbox', 'slug'), '20081226' );
         $scripts->localize( 'post', 'postL10n', array(
             'tagsUsed' =>  __('Tags used on this post:'),
@@ -206 +206 @@
             'l10n_print_after' => 'try{convertEntities(postL10n);}catch(e){};'
         ) );
-        $scripts->add( 'page', '/wp-admin/js/page.js', array('jquery', 'slug', 'wp-lists', 'postbox'), '20081210' );
+        $scripts->add( 'page', '/wp-admin/js/page.js', array('jquery', 'slug', 'wp-lists', 'postbox'), '20081226' );
         $scripts->localize( 'page', 'postL10n', array(
             'cancel' => __('Cancel'),
@@ -227 +227 @@
             'l10n_print_after' => 'try{convertEntities(postL10n);}catch(e){};'
         ) );
-        $scripts->add( 'link', '/wp-admin/js/link.js', array('jquery-ui-tabs', 'wp-lists', 'postbox'), '20081210' );
-        $scripts->add( 'comment', '/wp-admin/js/comment.js', array('jquery'), '20081210' );
+        $scripts->add( 'link', '/wp-admin/js/link.js', array('jquery-ui-tabs', 'wp-lists', 'postbox'), '20081226' );
+        $scripts->add( 'comment', '/wp-admin/js/comment.js', array('jquery'), '20081226' );
         $scripts->localize( 'comment', 'commentL10n', array(
             'cancel' => __('Cancel'),
@@ -263 +263 @@
         $scripts->add( 'theme-preview', '/wp-admin/js/theme-preview.js', array( 'thickbox', 'jquery' ), '20081210' );
 
-        $scripts->add( 'inline-edit-post', '/wp-admin/js/inline-edit-post.js', array( 'jquery', 'jquery-form', 'suggest' ), '20081210' );
+        $scripts->add( 'inline-edit-post', '/wp-admin/js/inline-edit-post.js', array( 'jquery', 'jquery-form', 'suggest' ), '20081226' );
         $scripts->localize( 'inline-edit-post', 'inlineEditL10n', array(
             'error' => __('Error while saving the changes.'),
@@ -271 +271 @@
         ) );
 
-        $scripts->add( 'inline-edit-tax', '/wp-admin/js/inline-edit-tax.js', array( 'jquery', 'jquery-form' ), '20081210' );
+        $scripts->add( 'inline-edit-tax', '/wp-admin/js/inline-edit-tax.js', array( 'jquery', 'jquery-form' ), '20081226' );
         $scripts->localize( 'inline-edit-tax', 'inlineEditL10n', array(
             'error' => __('Error while saving the changes.'),
@@ -285 +285 @@
         $scripts->add( 'farbtastic', '/wp-admin/js/farbtastic.js', array('jquery'), '1.2' );
 
-        $scripts->add( 'dashboard', '/wp-admin/js/dashboard.js', array( 'jquery', 'admin-comments', 'postbox' ), '20081210' );
+        $scripts->add( 'dashboard', '/wp-admin/js/dashboard.js', array( 'jquery', 'admin-comments', 'postbox' ), '20081226' );
 
         $scripts->add( 'hoverIntent', '/wp-includes/js/hoverIntent.js', array('jquery'), '20081210' );
@@ -322 +322 @@
 
     $styles->add( 'ie', '/wp-admin/css/ie.css', array(), '20081210' );
-    $styles->add_data( 'ie', 'conditional', 'gte IE 6' );
+    $styles->add_data( 'ie', 'conditional', 'lte IE 7' );
 
     $styles->add( 'colors', true, array(), '20081210' ); // Register "meta" stylesheet for admin colors
@@ -330 +330 @@
     $styles->add_data( 'colors-classic', 'rtl', true );
 
-    $styles->add( 'global', '/wp-admin/css/global.css', array(), '20081210' );
+    $styles->add( 'global', '/wp-admin/css/global.css', array(), '20081226' );
     $styles->add( 'media', '/wp-admin/css/media.css', array(), '20081210' );
     $styles->add( 'widgets', '/wp-admin/css/widgets.css', array(), '20081210' );
@@ -341 +341 @@
     $styles->add( 'plugin-install', '/wp-admin/css/plugin-install.css', array(), '20081210' );
     $styles->add( 'farbtastic', '/wp-admin/css/farbtastic.css', array(), '1.2' );
+    $styles->add( 'no-js', '/wp-admin/css/no-js.css', array(), '20081225' );
 
     foreach ( $rtl_styles as $rtl_style )