Make WordPress Core

Changeset 29613


Ignore:
Timestamp:
08/26/2014 04:21:06 AM (10 years ago)
Author:
wonderboymusic
Message:

Media Modal:

  • the Image Details frame does not have a menu, indicate that by setting menu: false when adding states.
  • mimic the default router rendering by toggling a hide-menu class on the frame when the region for menu is null
  • remove the dead render code for the Image Details frame

See #29339.

Location:
trunk/src/wp-includes
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-includes/css/media-views.css

    r29607 r29613  
    17801780}
    17811781
    1782 .image-details .media-frame-menu {
    1783     display: none;
    1784 }
    1785 
    17861782.image-details .media-frame-title,
    17871783.image-details .media-frame-content,
  • trunk/src/wp-includes/js/media-views.js

    r29612 r29613  
    463463                view;
    464464
     465            this.frame.$el.toggleClass( 'hide-menu', ! mode );
    465466            if ( ! mode ) {
    466467                return;
     
    792793            // Initial region modes.
    793794            content:  'image-details',
    794             menu:     'image-details',
     795            menu:     false,
    795796            router:   false,
    796797            toolbar:  'image-details',
     
    12451246            // Region mode defaults.
    12461247            toolbar:       'replace',
     1248            menu:          false,
    12471249
    12481250            priority:      60,
     
    29702972            this.on( 'content:create:image-details', this.imageDetailsContent, this );
    29712973            this.on( 'content:render:edit-image', this.editImageContent, this );
    2972             this.on( 'menu:render:image-details', this.renderMenu, this );
    29732974            this.on( 'toolbar:render:image-details', this.renderImageDetailsToolbar, this );
    29742975            // override the select toolbar
     
    29802981                new media.controller.ImageDetails({
    29812982                    image: this.image,
    2982                     editable: false,
    2983                     menu: 'image-details'
     2983                    editable: false
    29842984                }),
    29852985                new media.controller.ReplaceImage({
     
    29892989                    multiple:  false,
    29902990                    title:     l10n.imageReplaceTitle,
    2991                     menu: 'image-details',
    29922991                    toolbar: 'replace',
    29932992                    priority:  80,
     
    30243023            // after bringing in the frame, load the actual editor via an ajax call
    30253024            view.loadEditor();
    3026 
    3027         },
    3028 
    3029         renderMenu: function( view ) {
    3030             var lastState = this.lastState(),
    3031                 previous = lastState && lastState.id,
    3032                 frame = this;
    3033 
    3034             view.set({
    3035                 cancel: {
    3036                     text:     l10n.imageDetailsCancel,
    3037                     priority: 20,
    3038                     click:    function() {
    3039                         if ( previous ) {
    3040                             frame.setState( previous );
    3041                         } else {
    3042                             frame.close();
    3043                         }
    3044                     }
    3045                 },
    3046                 separateCancel: new media.View({
    3047                     className: 'separator',
    3048                     priority: 40
    3049                 })
    3050             });
    30513025
    30523026        },
Note: See TracChangeset for help on using the changeset viewer.