From 5e19d905f8e946a10a634ce1ed6d26c6f1e5f4ff Mon Sep 17 00:00:00 2001 From: Zac Pullar-Strecker Date: Fri, 20 Jan 2017 11:11:47 +1300 Subject: [PATCH] Fix loading indicator position (fixes #6153) Fix through a different element target in js. This is untested because node_modules is hopelessly broken on my machine. --- admin/client/dist/js/bundle.js | 8 +++++--- admin/client/src/legacy/LeftAndMain.js | 2 +- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/admin/client/dist/js/bundle.js b/admin/client/dist/js/bundle.js index a2154c441..04c518826 100644 --- a/admin/client/dist/js/bundle.js +++ b/admin/client/dist/js/bundle.js @@ -425,7 +425,7 @@ case"alpha":return u["default"].isAlpha(e) case"regex":return u["default"].matches(e,n.pattern) case"max":return e.length<=n.length case"email":return u["default"].isEmail(e) -default:console.warn("Unknown validation rule used: '"+t+"'")}}},{key:"validateFieldSchema",value:function l(e){return this.validateField(e.name,e.validation,null!==e.leftTitle?e.leftTitle:e.title,e.customValidationMessage) +default:return console.warn("Unknown validation rule used: '"+t+"'"),!1}}},{key:"validateFieldSchema",value:function l(e){return this.validateField(e.name,e.validation,null!==e.leftTitle?e.leftTitle:e.title,e.customValidationMessage) }},{key:"getMessage",value:function d(e,t){var n="" if("string"==typeof t.message)n=t.message @@ -1128,8 +1128,10 @@ minHeight:370,maxHeight:400,closeOnEscape:!1,open:function e(){$(".ui-widget-ove }})},onunmatch:function x(){this._super()},open:function R(){this.ssdialog("open")},close:function I(){this.ssdialog("close")},toggle:function F(e){this.is(":visible")?this.close():this.open()},reauthenticate:function A(e){ "undefined"!=typeof e.SecurityID&&$(":input[name=SecurityID]").val(e.SecurityID),"undefined"!=typeof e.TempID&&$("body").data("member-tempid",e.TempID),this.close()}}),$("form.loading,.cms-content.loading,.cms-content-fields.loading,.cms-content-view.loading").entwine({ -onmatch:function D(){this.append('
'),this._super()},onunmatch:function M(){this.find(".cms-content-loading-overlay,.cms-content-loading-spinner").remove(), -this._super()}}),$(".cms .cms-panel-link").entwine({onclick:function N(e){if($(this).hasClass("external-link"))return void e.stopPropagation() +onmatch:function D(){this.closest("div").append('
'),this._super()},onunmatch:function M(){ +this.find(".cms-content-loading-overlay,.cms-content-loading-spinner").remove(),this._super()}}),$(".cms .cms-panel-link").entwine({onclick:function N(e){if($(this).hasClass("external-link"))return void e.stopPropagation() + + var t=this.attr("href"),n=t&&!t.match(/^#/)?t:this.data("href"),i={pjax:this.data("pjaxTarget")} $(".cms-container").loadPanel(n,null,i),e.preventDefault()}}),$(".cms .ss-ui-button-ajax").entwine({onclick:function onclick(e){$(this).removeClass("ui-button-text-only"),$(this).addClass("ss-ui-button-loading ui-button-text-icons") diff --git a/admin/client/src/legacy/LeftAndMain.js b/admin/client/src/legacy/LeftAndMain.js index b2fba66d5..71eed4bf0 100644 --- a/admin/client/src/legacy/LeftAndMain.js +++ b/admin/client/src/legacy/LeftAndMain.js @@ -1041,7 +1041,7 @@ $.entwine('ss', function($) { */ $('form.loading,.cms-content.loading,.cms-content-fields.loading,.cms-content-view.loading').entwine({ onmatch: function() { - this.append('
'); + this.closest('div').append('
'); this._super(); }, onunmatch: function() {