Changeset 2373

Timestamp:

01/19/2010 09:43:11 PM (15 years ago)

Author:

apeatling

Message:

Moving terms out of the JS so that they can be translated.

Location:

trunk

Files:

• bp-core/bp-core-adminbar.php (modified) (1 diff)
• bp-themes/bp-default/_inc/global.js (modified) (3 diffs)
• bp-themes/bp-default/functions.php (modified) (1 diff)

trunk/bp-core/bp-core-adminbar.php

@@ -21 +21 @@
 
     echo '</ul>';
-    echo '</div></div>';
+    echo "</div></div><!-- #wp-admin-bar -->\n\n";
 }
 

trunk/bp-themes/bp-default/_inc/global.js

@@ -158 +158 @@
                 if ( 'fav' == type ) {
                     if ( !j('div.item-list-tabs li#activity-favorites').length )
-                        j('div.item-list-tabs ul li#activity-atme').before( '<li id="activity-favorites"><a href="#">My Favorites (<span>0</span>)</a></li>');
+                        j('div.item-list-tabs ul li#activity-atme').before( '<li id="activity-favorites"><a href="#">' + bp_terms_my_favs + ' (<span>0</span>)</a></li>');
 
                     target.removeClass('fav');
@@ -517 +517 @@
 
                     if ( !i )
-                        j(this).before( '<li class="show-all"><a href="#' + parent_li.attr('id') + '/show-all/" title="Show all comments for this thread">Show all ' +  comment_count + ' comments</a></li>' );
+                        j(this).before( '<li class="show-all"><a href="#' + parent_li.attr('id') + '/show-all/" title="' + bp_terms_show_all_comments + '">' + bp_terms_show_all + ' ' + comment_count + ' ' + bp_terms_comments + '</a></li>' );
                 }
             });
@@ -806 +806 @@
                 button.fadeOut( 100, function() {
                     if ( j(this).hasClass('accept') )
-                        j(this).html( 'Accepted' ).fadeIn(50);
+                        j(this).html( bp_terms_accepted ).fadeIn(50);
                     else
-                        j(this).html( 'Rejected' ).fadeIn(50);
+                        j(this).html( bp_terms_rejected ).fadeIn(50);
                 });
             }

trunk/bp-themes/bp-default/functions.php

@@ -201 +201 @@
 }
 
+/* Add words that we need to use in JS to the end of the page so they can be translated and still used. */
+function bp_dtheme_js_terms() { ?>
+<script type="text/javascript">
+    var bp_terms_my_favs = '<?php _e( "My Favorites", "buddypress" ) ?>';
+    var bp_terms_accepted = '<?php _e( "Accepted", "buddypress" ) ?>';
+    var bp_terms_rejected = '<?php _e( "Rejected", "buddypress" ) ?>';
+    var bp_terms_show_all_comments = '<?php _e( "Show all comments for this thread", "buddypress" ) ?>';
+    var bp_terms_show_all = '<?php _e( "Show all", "buddypress" ) ?>';
+    var bp_terms_comments = '<?php _e( "comments", "buddypress" ) ?>';
+    </script>
+<?php
+}
+add_action( 'wp_footer', 'bp_dtheme_js_terms' );
+
 /* Show a notice when the theme is activated - workaround by Ozh (http://old.nabble.com/Activation-hook-exist-for-themes--td25211004.html) */
 if ( is_admin() && isset($_GET['activated'] ) && $pagenow == "themes.php" ) {