Changeset 2810 for branches/plugin/bbp-themes/bbp-twentyten/form-bbp_split.php

Timestamp:

01/17/2011 05:26:05 AM (15 years ago)

Author:

johnjamesjacoby

Message:

Normalize bbp-twentyten theme files. Introduce bbp_tab_index functions for forms. Rename some page- files to action- since they are not page templates. Introduce bbp_login and bbp_register place-holders in bbp-twentyten theme.

File:

• branches/plugin/bbp-themes/bbp-twentyten/form-bbp_split.php (modified) (5 diffs)

branches/plugin/bbp-themes/bbp-twentyten/form-bbp_split.php

@@ -2 +2 @@
 
 /**
- * Split topic form
+ * Split Topic
  *
  * @package bbPress
- * @subpackage Themes
+ * @subpackage Theme
  */
 
 ?>
+
 <?php if ( is_user_logged_in() && current_user_can( 'edit_topic', bbp_get_topic_id() ) ) : ?>
 
@@ -31 +32 @@
                     </div>
 
-                    <?php // @todo Make a codex and add the merge topic docs. ?>
-                    <?php // printf( __( 'For more information, check <a href="%s">this documentation.', 'bbpress' ), 'http://codex.bbpress.org/Merge_Topics' ); ?>
+                    <?php
+                    /**
+                     * @todo Make a codex and add the merge topic docs.
+                     * printf( __( 'For more information, check <a href="%s">this documentation.', 'bbpress' ), 'http://codex.bbpress.org/Merge_Topics' );
+                     */
+                    ?>
 
                     <fieldset>
@@ -38 +43 @@
 
                         <div>
-                            <input name="bbp_topic_split_option" id="bbp_topic_split_option_reply" type="radio" checked="checked" value="reply" tabindex="10" />
+                            <input name="bbp_topic_split_option" id="bbp_topic_split_option_reply" type="radio" checked="checked" value="reply" tabindex="<?php bbp_tab_index(); ?>" />
                             <label for="bbp_topic_split_option_reply"><?php _e( 'Creating a new topic in this forum:', 'bbpress' ); ?></label>
-                            <input type="text" id="bbp_topic_split_destination_title" value="<?php bbp_topic_title(); ?>" tabindex="12" size="40" name="bbp_topic_split_destination_title" /><br />
+                            <input type="text" id="bbp_topic_split_destination_title" value="<?php bbp_topic_title(); ?>" tabindex="<?php bbp_tab_index(); ?>" size="40" name="bbp_topic_split_destination_title" /><br />
 
-                            <input name="bbp_topic_split_option" id="bbp_topic_split_option_existing" type="radio" value="existing" tabindex="14" />
+                            <input name="bbp_topic_split_option" id="bbp_topic_split_option_existing" type="radio" value="existing" tabindex="<?php bbp_tab_index(); ?>" />
                             <label for="bbp_topic_split_option_existing"><?php _e( 'Use an existing topic in this forum:', 'bbpress' ); ?></label>
 
                             <?php
                                 global $bbp;
-                                bbp_dropdown( array( 'post_type' => $bbp->topic_id, 'post_parent' => bbp_get_topic_forum_id( bbp_get_topic_id() ), 'selected' => -1, 'exclude' => bbp_get_topic_id(), 'select_id' => 'bbp_destination_topic', 'tab' => 16, 'none_found' => __( 'No topics were found to which the topic could be split to!', 'bbpress' ) ) );
+                                bbp_dropdown( array( 'post_type' => $bbp->topic_id, 'post_parent' => bbp_get_topic_forum_id( bbp_get_topic_id() ), 'selected' => -1, 'exclude' => bbp_get_topic_id(), 'select_id' => 'bbp_destination_topic', 'none_found' => __( 'No topics were found to which the topic could be split to!', 'bbpress' ) ) );
                             ?>
 
@@ -60 +65 @@
                             <?php if ( bbp_is_subscriptions_active() ) : ?>
 
-                                <input name="bbp_topic_subscribers" id="bbp_topic_subscribers" type="checkbox" value="1" checked="checked" tabindex="4" />
+                                <input name="bbp_topic_subscribers" id="bbp_topic_subscribers" type="checkbox" value="1" checked="checked" tabindex="<?php bbp_tab_index(); ?>" />
                                 <label for="bbp_topic_subscribers"><?php _e( 'Copy subscribers to the new topic', 'bbpress' ); ?></label><br />
 
                             <?php endif; ?>
 
-                            <input name="bbp_topic_favoriters" id="bbp_topic_favoriters" type="checkbox" value="1" checked="checked" tabindex="6" />
+                            <input name="bbp_topic_favoriters" id="bbp_topic_favoriters" type="checkbox" value="1" checked="checked" tabindex="<?php bbp_tab_index(); ?>" />
                             <label for="bbp_topic_favoriters"><?php _e( 'Copy favoriters to the new topic', 'bbpress' ); ?></label><br />
 
-                            <input name="bbp_topic_tags" id="bbp_topic_tags" type="checkbox" value="1" checked="checked" tabindex="8" />
+                            <input name="bbp_topic_tags" id="bbp_topic_tags" type="checkbox" value="1" checked="checked" tabindex="<?php bbp_tab_index(); ?>" />
                             <label for="bbp_topic_tags"><?php _e( 'Copy topic tags to the new topic', 'bbpress' ); ?></label><br />
 
@@ -75 +80 @@
 
                     <p id="bbp_topic_submit_container">
-                        <button type="submit" tabindex="18" id="bbp_merge_topic_submit" name="bbp_merge_topic_submit"><?php _e( 'Submit', 'bbpress' ); ?></button>
+                        <button type="submit" tabindex="<?php bbp_tab_index(); ?>" id="bbp_merge_topic_submit" name="bbp_merge_topic_submit"><?php _e( 'Submit', 'bbpress' ); ?></button>
                     </p>
                 </div>