WordPress.org

Make WordPress Core

Ticket #28842: 28842.disable-autoselect.2.patch

File 28842.disable-autoselect.2.patch, 2.1 KB (added by adamsilverstein, 5 years ago)

minor JS standards fix

  • src/wp-includes/js/media-grid.js

     
    135135                                        content:            'browse',
    136136                                        toolbar:            'select',
    137137                                        contentUserSetting: false,
    138                                         filterable:         'all'
     138                                        filterable:         'all',
     139                                        autoSelect:         false
    139140                                })
    140141                        ]);
    141142                },
  • src/wp-includes/js/media-views.js

     
    594594                        searchable:         true,
    595595                        filterable:         false,
    596596                        sortable:           true,
    597 
     597                        autoSelect:         true,
    598598                        describe:           false,
    599599                        // Uses a user setting to override the content mode.
    600600                        contentUserSetting: true,
     
    751751                        if ( 'upload' === content.mode() ) {
    752752                                this.frame.content.mode('browse');
    753753                        }
    754                         this.get('selection').add( attachment );
    755754
     755                        if ( this.get( 'autoSelect' ) ) {
     756                                this.get( 'selection' ).add( attachment );
     757                        }
     758
    756759                        // Set focus back to where it goes when an attachment is selected.
    757760                        $( '.attachments-browser .attachments .attachment' ).first().focus();
    758761                },
     
    57195722                        };
    57205723
    57215724                        if ( media.view.settings.mediaTrash &&
    5722                                 this.controller.activeModes.where( { id: 'grid' } ).length ) {
     5725                                this.controller.isModeActive( 'grid' ) ) {
    57235726
    57245727                                filters.trash = {
    57255728                                        text:  l10n.trash,
     
    58355838                        // Feels odd to bring the global media library switcher into the Attachment
    58365839                        // browser view. Is this a use case for doAction( 'add:toolbar-items:attachments-browser', this.toolbar );
    58375840                        // which the controller can tap into and add this view?
    5838                         if ( this.controller.activeModes.where( { id: 'grid' } ).length ) {
     5841                        if ( this.controller.isModeActive( 'grid' ) ) {
    58395842                                LibraryViewSwitcher = media.View.extend({
    58405843                                        className: 'view-switch media-grid-view-switch',
    58415844                                        template: media.template( 'media-library-view-switcher')