WordPress.org

Make WordPress Core

Ticket #22613: 22613-option-b.refresh.diff​

File 22613-option-b.refresh.diff​, 3.3 KB (added by lessbloat, 17 months ago)
Line 
1Index: wp-includes/js/media-views.js
2===================================================================
3--- wp-includes/js/media-views.js       (revision 22929)
4+++ wp-includes/js/media-views.js       (working copy)
5@@ -2574,17 +2574,30 @@
6                },
7 
8                updateSetting: function( event ) {
9-                       var $setting = $( event.target ).closest('[data-setting]'),
10+                       var target = $( event.target ),
11+                               $setting = target.closest('[data-setting]'),
12+                               el = $('.' + target.attr('class')),
13+                               spinner = el.parent().find('.spinner'),
14                                setting, value;
15 
16                        if ( ! $setting.length )
17                                return;
18 
19+                       if ( !spinner.length )
20+                               spinner = el.before('<span class="spinner"></span>').parent().find('.spinner');
21+
22+                       spinner.show();
23+
24                        setting = $setting.data('setting');
25                        value   = event.target.value;
26 
27                        if ( this.model.get( setting ) !== value )
28-                               this.model.save( setting, value );
29+                               this.model.save( { setting : value }, { success: function () {
30+                                               setTimeout( function(){
31+                                                       spinner.hide();
32+                                               }, 500 );
33+                                       }
34+                               });
35                },
36 
37                updateAll: function() {
38@@ -3657,4 +3670,4 @@
39                        this.$('img').attr( 'src', this.model.get('url') );
40                }
41        });
42-}(jQuery));
43\ No newline at end of file
44+}(jQuery));
45Index: wp-includes/media.php
46===================================================================
47--- wp-includes/media.php       (revision 22929)
48+++ wp-includes/media.php       (working copy)
49@@ -1253,7 +1253,7 @@
50                'multipart'           => true,
51                'urlstream_upload'    => true,
52        );
53-
54+       
55        // Multi-file uploading doesn't currently work in iOS Safari,
56        // single-file allows the built-in camera to be used as source for images
57        if ( wp_is_mobile() )
58@@ -1678,22 +1678,22 @@
59                <# if ( 'image' === data.type ) { #>
60                        <label class="setting" data-setting="title">
61                                <span><?php _e('Title'); ?></span>
62-                               <input type="text" value="{{ data.title }}" />
63+                               <input class="setting-image-title" type="text" value="{{ data.title }}" />
64                        </label>
65                        <label class="setting" data-setting="caption">
66                                <span><?php _e('Caption'); ?></span>
67-                               <textarea
68+                               <textarea class="setting-image-caption"
69                                        placeholder="<?php esc_attr_e('Describe this image&hellip;'); ?>"
70                                        >{{ data.caption }}</textarea>
71                        </label>
72                        <label class="setting" data-setting="alt">
73                                <span><?php _e('Alt Text'); ?></span>
74-                               <input type="text" value="{{ data.alt }}" />
75+                               <input class="setting-image-alt" type="text" value="{{ data.alt }}" />
76                        </label>
77                <# } else { #>
78                        <label class="setting" data-setting="title">
79                                <span><?php _e('Title'); ?></span>
80-                               <input type="text" value="{{ data.title }}"
81+                               <input class="setting-media-title" type="text" value="{{ data.title }}"
82                                <# if ( 'video' === data.type ) { #>
83                                        placeholder="<?php esc_attr_e('Describe this video&hellip;'); ?>"
84                                <# } else if ( 'audio' === data.type ) { #>
85Index: wp-includes/css/media-views.css
86===================================================================
87--- wp-includes/css/media-views.css     (revision 22929)
88+++ wp-includes/css/media-views.css     (working copy)
89@@ -254,8 +254,18 @@
90        float: left;
91        width: 100%;
92        margin: 1px 0;
93+       position: relative;
94 }
95 
96+.media-sidebar .setting .spinner {
97+       float: none;
98+       left: 0;
99+       min-height: 0;
100+       position: absolute;
101+       top: 2px;
102+       width: auto;
103+}
104+
105 .media-sidebar .setting span {
106        min-width: 30%;
107        margin-right: 4%;