<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN"
"http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head><meta http-equiv="content-type" content="text/html; charset=utf-8" />
<title>[18295] trunk: Make sure the browser nag in shown first on the dashboard no matter what! See #17323</title>
</head>
<body>

<style type="text/css"><!--
#msg dl.meta { border: 1px #006 solid; background: #369; padding: 6px; color: #fff; }
#msg dl.meta dt { float: left; width: 6em; font-weight: bold; }
#msg dt:after { content:':';}
#msg dl, #msg dt, #msg ul, #msg li, #header, #footer, #logmsg { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt;  }
#msg dl a { font-weight: bold}
#msg dl a:link    { color:#fc3; }
#msg dl a:active  { color:#ff0; }
#msg dl a:visited { color:#cc6; }
h3 { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; font-weight: bold; }
#msg pre { overflow: auto; background: #ffc; border: 1px #fa0 solid; padding: 6px; }
#logmsg { background: #ffc; border: 1px #fa0 solid; padding: 1em 1em 0 1em; }
#logmsg p, #logmsg pre, #logmsg blockquote { margin: 0 0 1em 0; }
#logmsg p, #logmsg li, #logmsg dt, #logmsg dd { line-height: 14pt; }
#logmsg h1, #logmsg h2, #logmsg h3, #logmsg h4, #logmsg h5, #logmsg h6 { margin: .5em 0; }
#logmsg h1:first-child, #logmsg h2:first-child, #logmsg h3:first-child, #logmsg h4:first-child, #logmsg h5:first-child, #logmsg h6:first-child { margin-top: 0; }
#logmsg ul, #logmsg ol { padding: 0; list-style-position: inside; margin: 0 0 0 1em; }
#logmsg > ul, #logmsg > ol { margin-left: 0; margin: 0 0 1em 0; }
#logmsg pre { background: #eee; padding: 1em; }
#logmsg blockquote { border: 1px solid #fa0; border-left-width: 10px; padding: 1em 1em 0 1em; background: white;}
#logmsg dl { margin: 0; }
#logmsg dt { font-weight: bold; }
#logmsg dd { margin: 0; padding: 0 0 0.5em 0; }
#logmsg dd:before { content:'\00bb';}
#logmsg table { border-spacing: 0px; border-collapse: collapse; border-top: 4px solid #fa0; border-bottom: 1px solid #fa0; background: #fff; }
#logmsg table th { text-align: left; font-weight: normal; padding: 0.2em 0.5em; border-top: 1px dotted #fa0; }
#logmsg table td { text-align: right; border-top: 1px dotted #fa0; padding: 0.2em 0.5em; }
#logmsg table thead th { text-align: center; border-bottom: 1px solid #fa0; }
#logmsg table th.Corner { text-align: left; }
#logmsg hr { border: none 0; border-top: 2px dashed #fa0; height: 1px; }
#header, #footer { color: #fff; background: #636; border: 1px #300 solid; padding: 6px; }
#patch { width: 100%; }
#patch h4 {font-family: verdana,arial,helvetica,sans-serif;font-size:10pt;padding:8px;background:#369;color:#fff;margin:0;}
#patch .propset h4, #patch .binary h4 {margin:0;}
#patch pre {padding:0;line-height:1.2em;margin:0;}
#patch .diff {width:100%;background:#eee;padding: 0 0 10px 0;overflow:auto;}
#patch .propset .diff, #patch .binary .diff  {padding:10px 0;}
#patch span {display:block;padding:0 10px;}
#patch .modfile, #patch .addfile, #patch .delfile, #patch .propset, #patch .binary, #patch .copfile {border:1px solid #ccc;margin:10px 0;}
#patch ins {background:#dfd;text-decoration:none;display:block;padding:0 10px;}
#patch del {background:#fdd;text-decoration:none;display:block;padding:0 10px;}
#patch .lines, .info {color:#888;background:#fff;}
--></style>
<div id="msg">
<dl class="meta">
<dt>Revision</dt> <dd><a href="http://trac.wordpress.org/changeset/18295">18295</a></dd>
<dt>Author</dt> <dd>azaozz</dd>
<dt>Date</dt> <dd>2011-06-13 00:24:47 +0000 (Mon, 13 Jun 2011)</dd>
</dl>

<h3>Log Message</h3>
<pre>Make sure the browser nag in shown first on the dashboard no matter what! See <a href="http://trac.wordpress.org/ticket/17323">#17323</a></pre>

<h3>Modified Paths</h3>
<ul>
<li><a href="#trunkwpadminincludestemplatephp">trunk/wp-admin/includes/template.php</a></li>
<li><a href="#trunkwpadminjspostboxdevjs">trunk/wp-admin/js/postbox.dev.js</a></li>
<li><a href="#trunkwpadminjspostboxjs">trunk/wp-admin/js/postbox.js</a></li>
<li><a href="#trunkwpincludesscriptloaderphp">trunk/wp-includes/script-loader.php</a></li>
</ul>

</div>
<div id="patch">
<h3>Diff</h3>
<a id="trunkwpadminincludestemplatephp"></a>
<div class="modfile"><h4>Modified: trunk/wp-admin/includes/template.php (18294 => 18295)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/wp-admin/includes/template.php        2011-06-12 21:38:57 UTC (rev 18294)
+++ trunk/wp-admin/includes/template.php        2011-06-13 00:24:47 UTC (rev 18295)
</span><span class="lines">@@ -943,9 +943,17 @@
</span><span class="cx">                 // Grab the ones the user has manually sorted. Pull them out of their previous context/priority and into the one the user chose
</span><span class="cx">                 if ( !$already_sorted &amp;&amp; $sorted = get_user_option( &quot;meta-box-order_$page&quot; ) ) {
</span><span class="cx">                         foreach ( $sorted as $box_context =&gt; $ids )
</span><del>-                                foreach ( explode(',', $ids) as $id )
</del><ins>+                                $_ids = explode(',', $ids);
+
+                                if ( $k = array_search('dashboard_browser_nag', $_ids, true) ) { // it's ok if this is the first value, i.e. $k == 0
+                                        unset($_ids[$k]);
+                                        array_unshift($_ids, 'dashboard_browser_nag');
+                                }
+
+                                foreach ( $_ids as $id ) {
</ins><span class="cx">                                         if ( $id )
</span><span class="cx">                                                 add_meta_box( $id, null, null, $page, $box_context, 'sorted' );
</span><ins>+                                }
</ins><span class="cx">                 }
</span><span class="cx">                 $already_sorted = true;
</span><span class="cx"> 
</span></span></pre></div>
<a id="trunkwpadminjspostboxdevjs"></a>
<div class="modfile"><h4>Modified: trunk/wp-admin/js/postbox.dev.js (18294 => 18295)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/wp-admin/js/postbox.dev.js        2011-06-12 21:38:57 UTC (rev 18294)
+++ trunk/wp-admin/js/postbox.dev.js        2011-06-13 00:24:47 UTC (rev 18295)
</span><span class="lines">@@ -100,24 +100,18 @@
</span><span class="cx">                                 forcePlaceholderSize: true,
</span><span class="cx">                                 helper: 'clone',
</span><span class="cx">                                 opacity: 0.65,
</span><del>-                                start: function(e,ui) {
-                                        $('body').css({
-                                                WebkitUserSelect: 'none',
-                                                KhtmlUserSelect: 'none'
-                                        });
-                                        /*
-                                        if ( $.browser.msie )
-                                                return;
-                                        ui.item.addClass('noclick');
-                                        */
-                                },
</del><span class="cx">                                 stop: function(e,ui) {
</span><ins>+                                        if ( $(this).find('#dashboard_browser_nag').is(':visible') &amp;&amp; 'dashboard_browser_nag' != this.firstChild.id ) {
+                                                $(this).sortable('cancel');
+                                                return;
+                                        }
+
</ins><span class="cx">                                         postboxes.save_order(page);
</span><span class="cx">                                         ui.item.parent().removeClass('temp-border');
</span><del>-                                        $('body').css({
-                                                WebkitUserSelect: '',
-                                                KhtmlUserSelect: ''
-                                        });
</del><ins>+                                },
+                                receive: function(e,ui) {
+                                        if ( 'dashboard_browser_nag' == ui.item[0].id )
+                                                $(ui.sender).sortable('cancel');
</ins><span class="cx">                                 }
</span><span class="cx">                         });
</span><span class="cx">                 },
</span></span></pre></div>
<a id="trunkwpadminjspostboxjs"></a>
<div class="modfile"><h4>Modified: trunk/wp-admin/js/postbox.js (18294 => 18295)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/wp-admin/js/postbox.js        2011-06-12 21:38:57 UTC (rev 18294)
+++ trunk/wp-admin/js/postbox.js        2011-06-13 00:24:47 UTC (rev 18295)
</span><span class="lines">@@ -1 +1 @@
</span><del>-var postboxes;(function(a){postboxes={add_postbox_toggles:function(c,b){this.init(c,b);a(&quot;.postbox h3, .postbox .handlediv&quot;).click(function(){var e=a(this).parent(&quot;.postbox&quot;),f=e.attr(&quot;id&quot;);if(&quot;dashboard_browser_nag&quot;==f){return}e.toggleClass(&quot;closed&quot;);postboxes.save_state(c);if(f){if(!e.hasClass(&quot;closed&quot;)&amp;&amp;a.isFunction(postboxes.pbshow)){postboxes.pbshow(f)}else{if(e.hasClass(&quot;closed&quot;)&amp;&amp;a.isFunction(postboxes.pbhide)){postboxes.pbhide(f)}}}});a(&quot;.postbox h3 a&quot;).click(function(f){f.stopPropagation()});a(&quot;.postbox a.dismiss&quot;).click(function(g){var f=a(this).parents(&quot;.postbox&quot;).attr(&quot;id&quot;)+&quot;-hide&quot;;a(&quot;#&quot;+f).prop(&quot;checked&quot;,false).triggerHandler(&quot;click&quot;);return false});a(&quot;.hide-postbox-tog&quot;).click(function(){var e=a(this).val();if(a(this).prop(&quot;checked&quot;)){a(&quot;#&quot;+e).show();if(a.i
 sFunction(postboxes.pbshow)){postboxes.pbshow(e)}}else{a(&quot;#&quot;+e).hide();if(a.isFunction(postboxes.pbhide)){postboxes.pbhide(e)}}postboxes.save_state(c)});a('.columns-prefs input[type=&quot;radio&quot;]').click(function(){var e=a(this).val(),f,g,h=a(&quot;#poststuff&quot;);if(h.length){if(e==2){h.addClass(&quot;has-right-sidebar&quot;);a(&quot;#side-sortables&quot;).addClass(&quot;temp-border&quot;)}else{if(e==1){h.removeClass(&quot;has-right-sidebar&quot;);a(&quot;#normal-sortables&quot;).append(a(&quot;#side-sortables&quot;).children(&quot;.postbox&quot;))}}}else{for(f=4;(f&gt;e&amp;&amp;f&gt;1);f--){g=a(&quot;#&quot;+d(f)+&quot;-sortables&quot;);a(&quot;#&quot;+d(f-1)+&quot;-sortables&quot;).append(g.children(&quot;.postbox&quot;));g.parent().hide()}for(f=1;f&lt;=e;f++){g=a(&quot;#&quot;+d(f)+&quot;-sortables&quot;);if(g.parent().is(&quot;:hidden&quot;)){g.addClass(&quot;temp-border&quot;).parent().show()}}a(&quot;.postbox-container:visible&quot;).css(&quot;width&
 quot;,98/e+&quot;%&quot;)}postboxes.save_order(c)});function d(e){switch(e){case 1:return&quot;normal&quot;;break;case 2:return&quot;side&quot;;break;case 3:return&quot;column3&quot;;break;case 4:return&quot;column4&quot;;break;default:return&quot;&quot;}}},init:function(c,b){a.extend(this,b||{});a(&quot;#wpbody-content&quot;).css(&quot;overflow&quot;,&quot;hidden&quot;);a(&quot;.meta-box-sortables&quot;).sortable({placeholder:&quot;sortable-placeholder&quot;,connectWith:&quot;.meta-box-sortables&quot;,items:&quot;.postbox&quot;,handle:&quot;.hndle&quot;,cursor:&quot;move&quot;,distance:2,tolerance:&quot;pointer&quot;,forcePlaceholderSize:true,helper:&quot;clone&quot;,opacity:0.65,start:function(f,d){a(&quot;body&quot;).css({WebkitUserSelect:&quot;none&quot;,KhtmlUserSelect:&quot;none&quot;})},stop:function(f,d){postboxes.save_order(c);d.item.parent().removeClass(&quot;temp-border&quot;);a(&quot;body&quot;).css({WebkitUserSelect:&quot;&quot;,KhtmlUserSelect:&quot;&quot;})}})
 },save_state:function(d){var b=a(&quot;.postbox&quot;).filter(&quot;.closed&quot;).map(function(){return this.id}).get().join(&quot;,&quot;),c=a(&quot;.postbox&quot;).filter(&quot;:hidden&quot;).map(function(){return this.id}).get().join(&quot;,&quot;);a.post(ajaxurl,{action:&quot;closed-postboxes&quot;,closed:b,hidden:c,closedpostboxesnonce:jQuery(&quot;#closedpostboxesnonce&quot;).val(),page:d})},save_order:function(c){var b,d=a(&quot;.columns-prefs input:checked&quot;).val()||0;b={action:&quot;meta-box-order&quot;,_ajax_nonce:a(&quot;#meta-box-order-nonce&quot;).val(),page_columns:d,page:c};a(&quot;.meta-box-sortables&quot;).each(function(){b[&quot;order[&quot;+this.id.split(&quot;-&quot;)[0]+&quot;]&quot;]=a(this).sortable(&quot;toArray&quot;).join(&quot;,&quot;)});a.post(ajaxurl,b)},pbshow:false,pbhide:false}}(jQuery));
</del><span class="cx">\ No newline at end of file
</span><ins>+var postboxes;(function(a){postboxes={add_postbox_toggles:function(c,b){this.init(c,b);a(&quot;.postbox h3, .postbox .handlediv&quot;).click(function(){var e=a(this).parent(&quot;.postbox&quot;),f=e.attr(&quot;id&quot;);if(&quot;dashboard_browser_nag&quot;==f){return}e.toggleClass(&quot;closed&quot;);postboxes.save_state(c);if(f){if(!e.hasClass(&quot;closed&quot;)&amp;&amp;a.isFunction(postboxes.pbshow)){postboxes.pbshow(f)}else{if(e.hasClass(&quot;closed&quot;)&amp;&amp;a.isFunction(postboxes.pbhide)){postboxes.pbhide(f)}}}});a(&quot;.postbox h3 a&quot;).click(function(f){f.stopPropagation()});a(&quot;.postbox a.dismiss&quot;).click(function(g){var f=a(this).parents(&quot;.postbox&quot;).attr(&quot;id&quot;)+&quot;-hide&quot;;a(&quot;#&quot;+f).prop(&quot;checked&quot;,false).triggerHandler(&quot;click&quot;);return false});a(&quot;.hide-postbox-tog&quot;).click(function(){var e=a(this).val();if(a(this).prop(&quot;checked&quot;)){a(&quot;#&quot;+e).show();if(a.i
 sFunction(postboxes.pbshow)){postboxes.pbshow(e)}}else{a(&quot;#&quot;+e).hide();if(a.isFunction(postboxes.pbhide)){postboxes.pbhide(e)}}postboxes.save_state(c)});a('.columns-prefs input[type=&quot;radio&quot;]').click(function(){var e=a(this).val(),f,g,h=a(&quot;#poststuff&quot;);if(h.length){if(e==2){h.addClass(&quot;has-right-sidebar&quot;);a(&quot;#side-sortables&quot;).addClass(&quot;temp-border&quot;)}else{if(e==1){h.removeClass(&quot;has-right-sidebar&quot;);a(&quot;#normal-sortables&quot;).append(a(&quot;#side-sortables&quot;).children(&quot;.postbox&quot;))}}}else{for(f=4;(f&gt;e&amp;&amp;f&gt;1);f--){g=a(&quot;#&quot;+d(f)+&quot;-sortables&quot;);a(&quot;#&quot;+d(f-1)+&quot;-sortables&quot;).append(g.children(&quot;.postbox&quot;));g.parent().hide()}for(f=1;f&lt;=e;f++){g=a(&quot;#&quot;+d(f)+&quot;-sortables&quot;);if(g.parent().is(&quot;:hidden&quot;)){g.addClass(&quot;temp-border&quot;).parent().show()}}a(&quot;.postbox-container:visible&quot;).css(&quot;width&
 quot;,98/e+&quot;%&quot;)}postboxes.save_order(c)});function d(e){switch(e){case 1:return&quot;normal&quot;;break;case 2:return&quot;side&quot;;break;case 3:return&quot;column3&quot;;break;case 4:return&quot;column4&quot;;break;default:return&quot;&quot;}}},init:function(c,b){a.extend(this,b||{});a(&quot;#wpbody-content&quot;).css(&quot;overflow&quot;,&quot;hidden&quot;);a(&quot;.meta-box-sortables&quot;).sortable({placeholder:&quot;sortable-placeholder&quot;,connectWith:&quot;.meta-box-sortables&quot;,items:&quot;.postbox&quot;,handle:&quot;.hndle&quot;,cursor:&quot;move&quot;,distance:2,tolerance:&quot;pointer&quot;,forcePlaceholderSize:true,helper:&quot;clone&quot;,opacity:0.65,stop:function(f,d){if(a(this).find(&quot;#dashboard_browser_nag&quot;).is(&quot;:visible&quot;)&amp;&amp;&quot;dashboard_browser_nag&quot;!=this.firstChild.id){a(this).sortable(&quot;cancel&quot;);return}postboxes.save_order(c);d.item.parent().removeClass(&quot;temp-border&quot;)},receive:function(
 f,d){if(&quot;dashboard_browser_nag&quot;==d.item[0].id){a(d.sender).sortable(&quot;cancel&quot;)}}})},save_state:function(d){var b=a(&quot;.postbox&quot;).filter(&quot;.closed&quot;).map(function(){return this.id}).get().join(&quot;,&quot;),c=a(&quot;.postbox&quot;).filter(&quot;:hidden&quot;).map(function(){return this.id}).get().join(&quot;,&quot;);a.post(ajaxurl,{action:&quot;closed-postboxes&quot;,closed:b,hidden:c,closedpostboxesnonce:jQuery(&quot;#closedpostboxesnonce&quot;).val(),page:d})},save_order:function(c){var b,d=a(&quot;.columns-prefs input:checked&quot;).val()||0;b={action:&quot;meta-box-order&quot;,_ajax_nonce:a(&quot;#meta-box-order-nonce&quot;).val(),page_columns:d,page:c};a(&quot;.meta-box-sortables&quot;).each(function(){b[&quot;order[&quot;+this.id.split(&quot;-&quot;)[0]+&quot;]&quot;]=a(this).sortable(&quot;toArray&quot;).join(&quot;,&quot;)});a.post(ajaxurl,b)},pbshow:false,pbhide:false}}(jQuery));
</ins><span class="cx">\ No newline at end of file
</span></span></pre></div>
<a id="trunkwpincludesscriptloaderphp"></a>
<div class="modfile"><h4>Modified: trunk/wp-includes/script-loader.php (18294 => 18295)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/wp-includes/script-loader.php        2011-06-12 21:38:57 UTC (rev 18294)
+++ trunk/wp-includes/script-loader.php        2011-06-13 00:24:47 UTC (rev 18295)
</span><span class="lines">@@ -324,7 +324,7 @@
</span><span class="cx">                 $scripts-&gt;add( 'xfn', &quot;/wp-admin/js/xfn$suffix.js&quot;, array('jquery'), '20110524' );
</span><span class="cx">                 $scripts-&gt;add_data( 'xfn', 'group', 1 );
</span><span class="cx"> 
</span><del>-                $scripts-&gt;add( 'postbox', &quot;/wp-admin/js/postbox$suffix.js&quot;, array('jquery-ui-sortable'), '20110524' );
</del><ins>+                $scripts-&gt;add( 'postbox', &quot;/wp-admin/js/postbox$suffix.js&quot;, array('jquery-ui-sortable'), '20110612' );
</ins><span class="cx">                 $scripts-&gt;add_data( 'postbox', 'group', 1 );
</span><span class="cx"> 
</span><span class="cx">                 $scripts-&gt;add( 'post', &quot;/wp-admin/js/post$suffix.js&quot;, array('suggest', 'wp-lists', 'postbox'), '20110524' );
</span></span></pre>
</div>
</div>

</body>
</html>