From 0343f5056139c3bb4c41ce70594a7a5228ef7cb3 Mon Sep 17 00:00:00 2001 From: Weston Ruter Date: Mon, 20 Feb 2017 00:10:49 +0000 Subject: [PATCH] Customize: Prevent vertical clipping of thumbnail in header image customizer control. Removes some method overrides on `wp.customize.HeaderTool.ChoiceView` introduced in [27497]. See #21785. Props bradyvercher. Fixes #38559. Built from https://develop.svn.wordpress.org/trunk@40082 git-svn-id: http://core.svn.wordpress.org/trunk@40019 1a063a9b-81f0-0310-95a4-ce76da25c4cd --- wp-includes/js/customize-views.js | 42 --------------------------- wp-includes/js/customize-views.min.js | 2 +- wp-includes/version.php | 2 +- 3 files changed, 2 insertions(+), 44 deletions(-) diff --git a/wp-includes/js/customize-views.js b/wp-includes/js/customize-views.js index 83792cb1b6..29de08adcd 100644 --- a/wp-includes/js/customize-views.js +++ b/wp-includes/js/customize-views.js @@ -24,48 +24,10 @@ render: function() { this.$el.html(this.template(this.model.toJSON())); - this.setPlaceholder(); this.setButtons(); return this; }, - getHeight: function() { - var image = this.$el.find('img'), - saved, height, headerImageData; - - if (image.length) { - this.$el.find('.inner').hide(); - } else { - this.$el.find('.inner').show(); - return 40; - } - - saved = this.model.get('savedHeight'); - height = image.height() || saved; - - // happens at ready - if (!height) { - headerImageData = api.get().header_image_data; - - if (headerImageData && headerImageData.width && headerImageData.height) { - // hardcoded container width - height = 260 / headerImageData.width * headerImageData.height; - } - else { - // fallback for when no image is set - height = 40; - } - } - - return height; - }, - - setPlaceholder: function(_height) { - var height = _height || this.getHeight(); - this.model.set('savedHeight', height); - this.$el.height(height); - }, - setButtons: function() { var elements = $('#customize-control-header_image .actions .remove'); if (this.model.get('choice')) { @@ -132,10 +94,6 @@ }); }, - getHeight: api.HeaderTool.CurrentView.prototype.getHeight, - - setPlaceholder: api.HeaderTool.CurrentView.prototype.setPlaceholder, - select: function() { this.preventJump(); this.model.save(); diff --git a/wp-includes/js/customize-views.min.js b/wp-includes/js/customize-views.min.js index fad8c4766d..abba5ea142 100644 --- a/wp-includes/js/customize-views.min.js +++ b/wp-includes/js/customize-views.min.js @@ -1 +1 @@ -!function(a,b,c){if(b&&b.customize){var d=b.customize;d.HeaderTool.CurrentView=b.Backbone.View.extend({template:b.template("header-current"),initialize:function(){this.listenTo(this.model,"change",this.render),this.render()},render:function(){return this.$el.html(this.template(this.model.toJSON())),this.setPlaceholder(),this.setButtons(),this},getHeight:function(){var a,b,c,e=this.$el.find("img");return e.length?(this.$el.find(".inner").hide(),a=this.model.get("savedHeight"),b=e.height()||a,b||(c=d.get().header_image_data,b=c&&c.width&&c.height?260/c.width*c.height:40),b):(this.$el.find(".inner").show(),40)},setPlaceholder:function(a){var b=a||this.getHeight();this.model.set("savedHeight",b),this.$el.height(b)},setButtons:function(){var b=a("#customize-control-header_image .actions .remove");this.model.get("choice")?b.show():b.hide()}}),d.HeaderTool.ChoiceView=b.Backbone.View.extend({template:b.template("header-choice"),className:"header-view",events:{"click .choice,.random":"select","click .close":"removeImage"},initialize:function(){var a=[this.model.get("header").url,this.model.get("choice")];this.listenTo(this.model,"change:selected",this.toggleSelected),c.contains(a,d.get().header_image)&&d.HeaderTool.currentHeader.set(this.extendedModel())},render:function(){return this.$el.html(this.template(this.extendedModel())),this.toggleSelected(),this},toggleSelected:function(){this.$el.toggleClass("selected",this.model.get("selected"))},extendedModel:function(){var a=this.model.get("collection");return c.extend(this.model.toJSON(),{type:a.type})},getHeight:d.HeaderTool.CurrentView.prototype.getHeight,setPlaceholder:d.HeaderTool.CurrentView.prototype.setPlaceholder,select:function(){this.preventJump(),this.model.save(),d.HeaderTool.currentHeader.set(this.extendedModel())},preventJump:function(){var b=a(".wp-full-overlay-sidebar-content"),d=b.scrollTop();c.defer(function(){b.scrollTop(d)})},removeImage:function(a){a.stopPropagation(),this.model.destroy(),this.remove()}}),d.HeaderTool.ChoiceListView=b.Backbone.View.extend({initialize:function(){this.listenTo(this.collection,"add",this.addOne),this.listenTo(this.collection,"remove",this.render),this.listenTo(this.collection,"sort",this.render),this.listenTo(this.collection,"change",this.toggleList),this.render()},render:function(){this.$el.empty(),this.collection.each(this.addOne,this),this.toggleList()},addOne:function(a){var b;a.set({collection:this.collection}),b=new d.HeaderTool.ChoiceView({model:a}),this.$el.append(b.render().el)},toggleList:function(){var a=this.$el.parents().prev(".customize-control-title"),b=this.$el.find(".random").parent();this.collection.shouldHideTitle()?a.add(b).hide():a.add(b).show()}}),d.HeaderTool.CombinedList=b.Backbone.View.extend({initialize:function(a){this.collections=a,this.on("all",this.propagate,this)},propagate:function(a,b){c.each(this.collections,function(c){c.trigger(a,b)})}})}}(jQuery,window.wp,_); \ No newline at end of file +!function(a,b,c){if(b&&b.customize){var d=b.customize;d.HeaderTool.CurrentView=b.Backbone.View.extend({template:b.template("header-current"),initialize:function(){this.listenTo(this.model,"change",this.render),this.render()},render:function(){return this.$el.html(this.template(this.model.toJSON())),this.setButtons(),this},setButtons:function(){var b=a("#customize-control-header_image .actions .remove");this.model.get("choice")?b.show():b.hide()}}),d.HeaderTool.ChoiceView=b.Backbone.View.extend({template:b.template("header-choice"),className:"header-view",events:{"click .choice,.random":"select","click .close":"removeImage"},initialize:function(){var a=[this.model.get("header").url,this.model.get("choice")];this.listenTo(this.model,"change:selected",this.toggleSelected),c.contains(a,d.get().header_image)&&d.HeaderTool.currentHeader.set(this.extendedModel())},render:function(){return this.$el.html(this.template(this.extendedModel())),this.toggleSelected(),this},toggleSelected:function(){this.$el.toggleClass("selected",this.model.get("selected"))},extendedModel:function(){var a=this.model.get("collection");return c.extend(this.model.toJSON(),{type:a.type})},select:function(){this.preventJump(),this.model.save(),d.HeaderTool.currentHeader.set(this.extendedModel())},preventJump:function(){var b=a(".wp-full-overlay-sidebar-content"),d=b.scrollTop();c.defer(function(){b.scrollTop(d)})},removeImage:function(a){a.stopPropagation(),this.model.destroy(),this.remove()}}),d.HeaderTool.ChoiceListView=b.Backbone.View.extend({initialize:function(){this.listenTo(this.collection,"add",this.addOne),this.listenTo(this.collection,"remove",this.render),this.listenTo(this.collection,"sort",this.render),this.listenTo(this.collection,"change",this.toggleList),this.render()},render:function(){this.$el.empty(),this.collection.each(this.addOne,this),this.toggleList()},addOne:function(a){var b;a.set({collection:this.collection}),b=new d.HeaderTool.ChoiceView({model:a}),this.$el.append(b.render().el)},toggleList:function(){var a=this.$el.parents().prev(".customize-control-title"),b=this.$el.find(".random").parent();this.collection.shouldHideTitle()?a.add(b).hide():a.add(b).show()}}),d.HeaderTool.CombinedList=b.Backbone.View.extend({initialize:function(a){this.collections=a,this.on("all",this.propagate,this)},propagate:function(a,b){c.each(this.collections,function(c){c.trigger(a,b)})}})}}(jQuery,window.wp,_); \ No newline at end of file diff --git a/wp-includes/version.php b/wp-includes/version.php index 8aa68d0579..ee497fcc36 100644 --- a/wp-includes/version.php +++ b/wp-includes/version.php @@ -4,7 +4,7 @@ * * @global string $wp_version */ -$wp_version = '4.8-alpha-40080'; +$wp_version = '4.8-alpha-40082'; /** * Holds the WordPress DB revision, increments when changes are made to the WordPress DB schema.