WordPress.org
Get WordPress

Make WordPress Core

Changeset 13788


Ignore:
Timestamp:
03/21/2010 06:06:18 AM (16 years ago)
Author:
dd32
Message:

Cleanup of Custom Background. Move JS to external file, Add @since, Remove redundant steps functionality, Preview of Image alignment. See #12186

Location:
trunk
Files:
2 added
2 edited

Legend:

Unmodified
Added
Removed

  • trunk/wp-admin/custom-background.php

    r13765 r13788  
    11<?php
    22/**
    3  * The custom background image script.
     3 * The custom background script.
    44 *
    55 * @package WordPress
    … …  
    88
    99/**
    10  * The custom background image class.
     10 * The custom background class.
    1111 *
    12  * @since unknown
     12 * @since 3.0
    1313 * @package WordPress
    1414 * @subpackage Administration
    … …  
    2929         *
    3030         * @var callback
    31          * @since unknown
     31         * @since 3.0
    3232         * @access private
    3333         */
    … …  
    3737         * PHP4 Constructor - Register administration header callback.
    3838         *
    39          * @since unknown
     39         * @since 3.0
    4040         * @param callback $admin_header_callback
    4141         * @param callback $admin_image_div_callback Optional custom image div output callback.
    … …  
    5050         * Set up the hooks for the Custom Background admin page.
    5151         *
    52          * @since unknown
     52         * @since 3.0
    5353         */
    5454        function init() {
    … …  
    5858                $page = add_theme_page(__('Background'), __('Background'), 'switch_themes', 'custom-background', array(&$this, 'admin_page'));
    5959
    60                 add_action("admin_print_scripts-$page", array(&$this, 'js_includes'));
    61                 add_action("admin_print_styles-$page", array(&$this, 'css_includes'));
    62                 add_action("admin_head-$page", array(&$this, 'js'), 50);
    63                 add_action("admin_head-$page", array(&$this, 'take_action'), 49);
     60                add_action("load-$page", array(&$this, 'admin_load'));
     61                add_action("load-$page", array(&$this, 'take_action'), 49);
     62                add_action("load-$page", array(&$this, 'handle_upload'), 49);
     63
    6464                if ( $this->admin_header_callback )
    6565                        add_action("admin_head-$page", $this->admin_header_callback, 51);
    … …  
    6767
    6868        /**
    69          * Get the current step.
    70          *
    71          * @since unknown
    72          *
    73          * @return int Current step
    74          */
    75         function step() {
    76                 if ( ! isset( $_GET['step'] ) )
    77                         return 1;
    78 
    79                 $step = (int) $_GET['step'];
    80                 if ( $step < 1 || 3 < $step )
    81                         $step = 1;
    82 
    83                 return $step;
    84         }
    85 
    86         /**
    87          * Set up the enqueue for the JavaScript files.
    88          *
    89          * @since unknown
    90          */
    91         function js_includes() {
    92                 wp_enqueue_script('farbtastic');
    93         }
    94 
    95         /**
    96          * Set up the enqueue for the CSS files
    97          *
    98          * @since unknown
    99          */
    100         function css_includes() {
     69         * Set up the enqueue for the CSS & JavaScript files.
     70         *
     71         * @since 3.0
     72         */
     73        function admin_load() {
     74                wp_enqueue_script('custom-background');
    10175                wp_enqueue_style('farbtastic');
    10276        }
    … …  
    10579         * Execute custom background modification.
    10680         *
    107          * @since unknown
     81         * @since 3.0
    10882         */
    10983        function take_action() {
    110                 if ( ! current_user_can('switch_themes') )
    111                         return;
    11284
    11385                if ( empty($_POST) )
    … …  
    11688                check_admin_referer('custom-background');
    11789
    118                 if ( isset($_POST['reset-background']) )
     90                // @TODO: No UI entry point for this:
     91                if ( isset($_POST['reset-background']) ) {
    11992                        remove_theme_mods();
     93                        return;
     94                }
     95                if ( isset($_POST['remove-background']) ) {
     96                        // @TODO: Uploaded files are not removed here.
     97                        set_theme_mod('background_image', '');
     98                }
     99
    120100                if ( isset($_POST['background-repeat']) ) {
    121101                        if ( in_array($_POST['background-repeat'], array('repeat', 'no-repeat')) )
    … …  
    139119                        set_theme_mod('background_attachment', $attachment);
    140120                }
    141                 if ( isset($_POST['remove-background']) )
    142                         set_theme_mod('background_image', '');
    143                 if ( isset( $_POST['background-color'] ) ) {
     121                if ( isset($_POST['background-color']) ) {
    144122                        $color = preg_replace('/[^0-9a-fA-F]/', '', $_POST['background-color']);
    145123                        if ( strlen($color) == 6 || strlen($color) == 3 )
    … …  
    153131
    154132        /**
    155          * Execute Javascript depending on step.
    156          *
    157          * @since unknown
    158          */
    159         function js() {
    160                 $this->js_1();
    161         }
    162 
    163         /**
    164          * Display Javascript based on Step 1.
    165          *
    166          * @since unknown
    167          */
    168         function js_1() { ?>
    169 <script type="text/javascript">
    170         var buttons = ['#pickcolor'],
    171                 farbtastic;
    172 
    173         function pickColor(color) {
    174                 jQuery('#background-color').val(color);
    175                 farbtastic.setColor(color);
    176                 jQuery('#custom-background-image').css('background-color', color);
    177         }
    178 
    179         jQuery(document).ready(function() {
    180                 jQuery('#pickcolor').click(function() {
    181                         jQuery('#colorPickerDiv').show();
    182                 });
    183                 jQuery('#background-color').keyup(function() {
    184                         var _hex = jQuery('#background-color').val();
    185                         var hex = _hex;
    186                         if ( hex[0] != '#' )
    187                                 hex = '#' + hex;
    188                         hex = hex.replace(/[^#a-fA-F0-9]+/, '');
    189                         if ( hex != _hex )
    190                                 jQuery('#background-color').val(hex);
    191                         if ( hex.length == 4 || hex.length == 7 )
    192                                 pickColor( hex );
    193                 });
    194 
    195                 farbtastic = jQuery.farbtastic('#colorPickerDiv', function(color) { pickColor(color); });
    196                 pickColor('#<?php background_color(); ?>');
    197         });
    198 
    199         jQuery(document).mousedown(function(){
    200                 hide_picker(); // Make the picker disappear if you click outside its div element
    201         });
    202 
    203         function hide_picker(what) {
    204                 var update = false;
    205                 jQuery('#colorPickerDiv').each(function(){
    206                         var id = jQuery(this).attr('id');
    207                         if (id == what) {
    208                                 return;
    209                         }
    210                         var display = jQuery(this).css('display');
    211                         if (display == 'block') {
    212                                 jQuery(this).fadeOut(2);
    213                         }
    214                 });
    215         }
    216 
    217 </script>
    218 <?php
    219         }
    220 
    221         /**
    222          * Display first step of custom background image page.
    223          *
    224          * @since unknown
    225          */
    226         function step_1() {
     133         * Display the custom background page.
     134         *
     135         * @since 3.0
     136         */
     137        function admin_page() {
    227138?>
    228139<div class="wrap" id="custom-background">
    … …  
    240151        } else {
    241152                if ( $bgcolor = get_background_color() )
    242                         $bgcolor = ' style="background-color: #' . $bgcolor . ';"';
    243                 else
    244                         $bgcolor = '';
     153                        $bgcolor = 'background-color: #' . $bgcolor . ';';
     154
     155                if ( $align = get_theme_mod('background_position', 'left') )
     156                        $align = "text-align: $align;";         
    245157?>
    246 <div id="custom-background-image"<?php echo $bgcolor; ?>>
     158<div id="custom-background-image"  style="<?php echo $bgcolor, $align ?>">
    247159<?php if ( get_background_image() ) { ?>
    248160<img class="custom-background-image" src="<?php background_image(); ?>" />
    249161<?php } ?>
     162<br class="clear" />
    250163</div>
    251164<?php } ?>
    252165<h3><?php _e('Change Display Options') ?></h3>
    253 <form method="post" action="<?php echo esc_attr(add_query_arg('step', 1)) ?>">
     166<form method="post" action="">
    254167<table>
    255168<thead>
    … …  
    315228
    316229<h3><?php _e('Upload New Background Image'); ?></h3>
    317 <form enctype="multipart/form-data" id="uploadForm" method="POST" action="<?php echo esc_attr(add_query_arg('step', 2)) ?>">
     230<form enctype="multipart/form-data" id="uploadForm" method="POST" action="">
    318231<label for="upload"><?php _e('Choose an image from your computer:'); ?></label><br /><input type="file" id="upload" name="import" />
    319232<input type="hidden" name="action" value="save" />
    … …  
    327240<h3><?php _e('Remove Background Image'); ?></h3>
    328241<p><?php _e('This will remove the background image. You will not be able to retrieve any customizations.') ?></p>
    329 <form method="post" action="<?php echo esc_attr(add_query_arg('step', 1)) ?>">
     242<form method="post" action="">
    330243<?php wp_nonce_field('custom-background'); ?>
    331244<input type="submit" class="button" name="remove-background" value="<?php esc_attr_e('Remove Background'); ?>" />
    332245</form>
    333 
    334246<?php endif; ?>
     247
    335248</div>
    336249<?php
    … …  
    338251
    339252        /**
    340          * Display second step of custom background image page.
    341          *
    342          * @since unknown
    343          */
    344         function step_2() {
     253         * Handle a Image upload for the background image.
     254         *
     255         * @since 3.0
     256         */
     257        function handle_upload() {
     258
     259                if ( empty($_FILES) )
     260                        return;
     261
    345262                check_admin_referer('custom-background');
    346263                $overrides = array('test_form' => false);
    … …  
    372289                do_action('wp_create_file_in_uploads', $file, $id); // For replication
    373290                $this->updated = true;
    374                 return $this->finished();
    375         }
    376 
    377         /**
    378          * Display last step of custom header image page.
    379          *
    380          * @since unknown
    381          */
    382         function finished() {
    383                 $this->step_1();
    384         }
    385 
    386         /**
    387          * Display the page based on the current step.
    388          *
    389          * @since unknown
    390          */
    391         function admin_page() {
    392                 if ( ! current_user_can('switch_themes') )
    393                         wp_die(__('You do not have permission to customize the background.'));
    394                 $step = $this->step();
    395                 if ( 1 == $step )
    396                         $this->step_1();
    397                 elseif ( 2 == $step )
    398                         $this->step_2();
    399291        }
    400292

  • trunk/wp-includes/script-loader.php

    r13766 r13788  
    407407                        'warnDelete' => __( "You are about to permanently delete this menu. \n 'Cancel' to stop, 'OK' to delete." ),
    408408                ) );
     409
     410                $scripts->add( 'custom-background', "/wp-admin/js/custom-background$suffix.js", array('farbtastic'), '20100321' );
     411                $scripts->add_data( 'custom-background', 'group', 1 );
     412                // See wp_just_in_time_script_localization() for translation data for this object
     413
    409414        }
    410415}
    … …  
    510515
    511516/**
    512  * Load localized script just in time for MCE.
     517 * Load localized data on print rather than initialization.
    513518 *
    514519 * These localizations require information that may not be loaded even by init.
    … …  
    527532                'l10n_print_after' => 'try{convertEntities(autosaveL10n);}catch(e){};'
    528533        ) );
     534
     535        wp_localize_script( 'custom-background', 'customBackgroundL10n', array(
     536                'backgroundcolor' => '#' . get_background_color(),
     537        ) );
    529538}
    530539
Note: See TracChangeset for help on using the changeset viewer.