WordPress.org

Make WordPress Core

Ticket #25963: 25963.6.diff

File 25963.6.diff, 5.5 KB (added by adamsilverstein, 6 years ago)

simpler, remove unused routes, don't clutter history

  • src/wp-admin/js/theme.js

     
    292292                                scroll = document.body.scrollTop;
    293293
    294294                                // Clean the url structure
    295                                 themes.router.navigate( '' );
     295                                themes.router.navigate( themes.router.baseUrl( '' ), { replace: true } );
    296296
    297297                                // Restore scroll position
    298298                                document.body.scrollTop = scroll;
     
    380380themes.view.Themes = wp.Backbone.View.extend({
    381381
    382382        className: 'themes',
     383        $overlay: $( 'div.theme-overlay' ),
    383384
    384385        // Number to keep track of scroll position
    385386        // while in theme-overlay mode
     
    511512                this.model = self.collection.get( id );
    512513
    513514                // Trigger a route update for the current model
    514                 themes.router.navigate( 'theme/' + this.model.id );
     515                themes.router.navigate( themes.router.baseUrl( '?theme=' + this.model.id ), { replace: true } );
    515516
    516517                // Sets this.view to 'detail'
    517518                this.setView( 'detail' );
     
    523524                });
    524525
    525526                this.overlay.render();
    526                 this.$el.append( this.overlay.el );
     527                this.$overlay.html( this.overlay.el );
    527528
    528529                this.overlay.screenshotGallery();
    529530
     
    565566                        this.overlay.closeOverlay();
    566567
    567568                        // Trigger a route update for the current model
    568                         // that renders the new theme's overlay
    569                         themes.router.navigate( 'theme/' + nextModel.id, { trigger: true } );
     569                        self.theme.trigger( 'theme:expand', nextModel.cid );
     570
    570571                }
    571572        },
    572573
     
    589590                        this.overlay.closeOverlay();
    590591
    591592                        // Trigger a route update for the current model
    592                         // that renders the new theme's overlay
    593                         themes.router.navigate( 'theme/' + previousModel.id, { trigger: true } );
     593                        self.theme.trigger( 'theme:expand', previousModel.cid );
     594
    594595                }
    595596        }
    596597});
     
    615616
    616617        // Runs a search on the theme collection.
    617618        search: function( event ) {
     619                var self = this;
     620
    618621                // Clear on escape.
    619622                if ( event.type === 'keyup' && event.which === 27 ) {
    620623                        event.target.value = '';
    621624                }
    622625
    623                 this.collection.doSearch( event.target.value );
     626                self.collection.doSearch( event.target.value );
    624627
    625628                // Update the URL hash
    626629                if ( event.target.value ) {
    627                         themes.router.navigate( 'search/' + event.target.value, { replace: true } );
     630                        themes.router.navigate( themes.router.baseUrl( '?search=' + event.target.value ), { replace: true } );
    628631                } else {
    629                         themes.router.navigate( '' );
     632                        themes.router.navigate( themes.router.baseUrl( '' ), { replace: true } );
    630633                }
    631634        }
    632635});
     
    635638// Listens to [theme] and [search] params
    636639themes.routes = Backbone.Router.extend({
    637640
    638         routes: {
    639                 'search/*query': 'search',
    640                 'theme/*slug': 'theme'
     641        initialize: function() {
     642                this.routes = _.object([
     643                ]);
    641644        },
    642645
     646        baseUrl: function( url ) {
     647                return themes.data.settings.root + url;
     648        },
     649
    643650        // Set the search input value based on url
    644651        search: function( query ) {
    645652                $( '.theme-search' ).val( query );
     653                _.debounce( self.themes.doSearch( query ), 200 );
    646654        }
    647655});
    648656
     
    667675        render: function() {
    668676                // Render results
    669677                this.view.render();
    670 
    671                 // Calls the routes functionality
    672678                this.routes();
    673679
    674                 // Set ups history with pushState and our root
    675                 Backbone.history.start({ root: themes.data.settings.root });
     680                // Set the initial theme
     681                if ( 'undefined' !== typeof themes.data.settings.theme && '' !== themes.data.settings.theme ){
     682                        this.view.view.theme.trigger( 'theme:expand', this.view.collection.findWhere( { id: themes.data.settings.theme } ) );
     683                }
     684
     685                // Set the initial search
     686                if ( 'undefined' !== typeof themes.data.settings.search && '' !== themes.data.settings.search ){
     687                        $( '.theme-search' ).val( themes.data.settings.search );
     688                        this.themes.doSearch( themes.data.settings.search );
     689                }
     690
     691                // Start the router if browser supports History API
     692                if ( window.history && window.history.pushState ) {
     693                        // Calls the routes functionality
     694                        Backbone.history.start({ pushState: true, silent: true });
     695                }
    676696        },
    677697
    678698        routes: function() {
     
    680700                // Bind to our global thx object
    681701                // so that the object is available to sub-views
    682702                themes.router = new themes.routes();
    683 
    684                 // Handles theme details route event
    685                 themes.router.on( 'route:theme', function( slug ) {
    686                         self.view.view.expand( slug );
    687                 });
    688 
    689                 // Handles search route event
    690                 themes.router.on( 'route:search', function( query ) {
    691                         self.themes.doSearch( query );
    692                 });
    693703        }
    694704};
    695705
  • src/wp-admin/themes.php

     
    9191} else {
    9292        $themes = wp_prepare_themes_for_js( array( wp_get_theme() ) );
    9393}
     94wp_reset_vars( array( 'theme', 'search' ) );
    9495
    9596wp_localize_script( 'theme', '_wpThemeSettings', array(
    9697        'themes'   => $themes,
     
    9899                'canInstall'    => ( ! is_multisite() && current_user_can( 'install_themes' ) ),
    99100                'installURI'    => ( ! is_multisite() && current_user_can( 'install_themes' ) ) ? admin_url( 'theme-install.php' ) : null,
    100101                'confirmDelete' => __( "Are you sure you want to delete this theme?\n\nClick 'Cancel' to go back, 'OK' to confirm the delete." ),
    101                 'root'          => admin_url( 'themes.php' ),
     102                'root'          => parse_url( admin_url( 'themes.php' ), PHP_URL_PATH ),
    102103                'extraRoutes'   => '',
     104                'theme'         => esc_html( $theme ),
     105                'search'        => esc_html( $search ),
     106
    103107        ),
    104108        'l10n' => array(
    105109                'addNew' => __( 'Add New Theme' ),
     
    180184?>
    181185
    182186<div class="theme-browser"></div>
     187<div class="theme-overlay"></div>
    183188
    184189<?php
    185190// List broken themes, if any.