From ba1011c28b78bcffc33df97579874f143a3f42be Mon Sep 17 00:00:00 2001 From: Alex Rudyy Date: Wed, 17 Sep 2014 16:47:06 +0000 Subject: QPID-6093: Address code review comments: 1) Display promptMessages in edit broker dialog 2) Fix issue with button disaperaing on dialog resize on editing VH/VHN and broker 3) Fix context name/value pair editing in Safari 4) Stop posting form data on pressing Cancel button in dialogs for editing VH/VHN and broker git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk@1625650 13f79535-47bb-0310-9956-ffa450edef68 --- .../addVirtualHostNodeAndVirtualHost.html | 8 ++--- .../src/main/java/resources/editBroker.html | 7 ++-- .../src/main/java/resources/editVirtualHost.html | 7 ++-- .../main/java/resources/editVirtualHostNode.html | 7 ++-- .../js/qpid/common/ContextVariablesEditor.js | 4 +-- .../src/main/java/resources/js/qpid/common/util.js | 38 +++++++++++++++++++++- .../management/addVirtualHostNodeAndVirtualHost.js | 10 ++++-- .../resources/js/qpid/management/editBroker.js | 8 ++++- .../js/qpid/management/editVirtualHost.js | 6 ++++ .../js/qpid/management/editVirtualHostNode.js | 6 ++++ 10 files changed, 82 insertions(+), 19 deletions(-) (limited to 'qpid/java/broker-plugins') diff --git a/qpid/java/broker-plugins/management-http/src/main/java/resources/addVirtualHostNodeAndVirtualHost.html b/qpid/java/broker-plugins/management-http/src/main/java/resources/addVirtualHostNodeAndVirtualHost.html index 685b3bced4..3e3e931829 100644 --- a/qpid/java/broker-plugins/management-http/src/main/java/resources/addVirtualHostNodeAndVirtualHost.html +++ b/qpid/java/broker-plugins/management-http/src/main/java/resources/addVirtualHostNodeAndVirtualHost.html @@ -20,7 +20,7 @@ -->
- +
@@ -99,10 +99,10 @@
- -
+
+
-
+
diff --git a/qpid/java/broker-plugins/management-http/src/main/java/resources/editBroker.html b/qpid/java/broker-plugins/management-http/src/main/java/resources/editBroker.html index 2516396d4d..85ca617c10 100644 --- a/qpid/java/broker-plugins/management-http/src/main/java/resources/editBroker.html +++ b/qpid/java/broker-plugins/management-http/src/main/java/resources/editBroker.html @@ -20,6 +20,7 @@
+
Name*:
@@ -107,11 +108,11 @@
- -
+
+
-
+
diff --git a/qpid/java/broker-plugins/management-http/src/main/java/resources/editVirtualHost.html b/qpid/java/broker-plugins/management-http/src/main/java/resources/editVirtualHost.html index 29229137bb..f4826016ec 100644 --- a/qpid/java/broker-plugins/management-http/src/main/java/resources/editVirtualHost.html +++ b/qpid/java/broker-plugins/management-http/src/main/java/resources/editVirtualHost.html @@ -20,6 +20,7 @@
+
NOTE: All changes will only take effect after Virtual Host restart.
Name*:
@@ -125,11 +126,11 @@
- -
+
+
-
+
diff --git a/qpid/java/broker-plugins/management-http/src/main/java/resources/editVirtualHostNode.html b/qpid/java/broker-plugins/management-http/src/main/java/resources/editVirtualHostNode.html index 8c2c82fdbd..46e76c31f0 100644 --- a/qpid/java/broker-plugins/management-http/src/main/java/resources/editVirtualHostNode.html +++ b/qpid/java/broker-plugins/management-http/src/main/java/resources/editVirtualHostNode.html @@ -19,6 +19,7 @@
+
NOTE: All changes will only take effect after Virtual Host Node restart.
Name*:
@@ -42,11 +43,11 @@
- -
+
+
-
+
diff --git a/qpid/java/broker-plugins/management-http/src/main/java/resources/js/qpid/common/ContextVariablesEditor.js b/qpid/java/broker-plugins/management-http/src/main/java/resources/js/qpid/common/ContextVariablesEditor.js index 4f384315ac..e5d341de91 100644 --- a/qpid/java/broker-plugins/management-http/src/main/java/resources/js/qpid/common/ContextVariablesEditor.js +++ b/qpid/java/broker-plugins/management-http/src/main/java/resources/js/qpid/common/ContextVariablesEditor.js @@ -77,8 +77,8 @@ function (util, xhr, declare, array, connect, lang, domConstruct, parser, query, var addButton = registry.byNode(addButtonNode); var deleteButton = registry.byNode(deleteButtonNode); var layout = [[ - { name: "Name", field: "name", width: "40%", editable: true }, - { name: 'Actual Value', field: 'actualValue', width: '30%', editable: true}, + { name: "Name", field: "name", width: "40%", editable: true, selectOnClick: false, type: dojox.grid.cells._Widget, widgetClass: dijit.form.TextBox }, + { name: 'Actual Value', field: 'actualValue', width: '30%', editable: true, selectOnClick: false, type: dojox.grid.cells._Widget, widgetClass: dijit.form.TextBox}, { name: 'Effective Value', field: 'effectiveValue', width: '30%', editable: false} ]]; var data = []; diff --git a/qpid/java/broker-plugins/management-http/src/main/java/resources/js/qpid/common/util.js b/qpid/java/broker-plugins/management-http/src/main/java/resources/js/qpid/common/util.js index ea011ad9dc..a439e0512a 100644 --- a/qpid/java/broker-plugins/management-http/src/main/java/resources/js/qpid/common/util.js +++ b/qpid/java/broker-plugins/management-http/src/main/java/resources/js/qpid/common/util.js @@ -30,6 +30,8 @@ define(["dojo/_base/xhr", "dojo/query", "dojo/parser", "dojo/store/Memory", + "dojo/window", + "dojo/on", "dojox/html/entities", "qpid/common/metadata", "qpid/common/widgetconfigurer", @@ -48,7 +50,7 @@ define(["dojo/_base/xhr", "dojox/validate/web", "dojo/domReady!" ], - function (xhr, array, event, lang, json, dom, geometry, domStyle, win, query, parser, Memory, entities, metadata, widgetconfigurer, registry) { + function (xhr, array, event, lang, json, dom, geometry, domStyle, win, query, parser, Memory, w, on, entities, metadata, widgetconfigurer, registry) { var util = {}; if (Array.isArray) { util.isArray = function (object) { @@ -735,5 +737,39 @@ define(["dojo/_base/xhr", return "^(([0-9a-zA-Z.-_]|::)+:[0-9]{1,5})|" + singleContextVarRegexp + "$"; } + util.resizeContentAreaAndRepositionDialog = function(contentNode, dialog) + { + var viewport = w.getBox(); + var contentDimension =dojo.position(contentNode); + var dialogDimension = dojo.position(dialog.domNode); + var dialogTitleAndFooterHeight = dialogDimension.h - contentDimension.h; + var dialogLeftRightSpaces = dialogDimension.w - contentDimension.w; + + var resize = function() + { + var viewport = w.getBox(); + var width = viewport.w * dialog.maxRatio; + var height = viewport.h * dialog.maxRatio; + var dialogDimension = dojo.position(dialog.domNode); + + var maxContentHeight = height - dialogTitleAndFooterHeight; + + // if width style is set on a dialog node, use dialog width + if (dialog.domNode.style && dialog.domNode.style.width) + { + width = dialogDimension.w; + } + var maxContentWidth = width - dialogLeftRightSpaces; + domStyle.set(contentNode, {"overflow": "auto", maxHeight: maxContentHeight + "px", maxWidth: maxContentWidth + "px"}); + + var dialogX = viewport.w/2 - dialogDimension.w/2; + var dialogY = viewport.h/2 - dialogDimension.h/2; + domStyle.set(dialog.domNode, {top: dialogY + "px", left: dialogX + "px"}); + dialog.resize(); + }; + resize(); + on(window, "resize", resize); + } + return util; }); diff --git a/qpid/java/broker-plugins/management-http/src/main/java/resources/js/qpid/management/addVirtualHostNodeAndVirtualHost.js b/qpid/java/broker-plugins/management-http/src/main/java/resources/js/qpid/management/addVirtualHostNodeAndVirtualHost.js index 96da8dc5fa..226674bf5a 100644 --- a/qpid/java/broker-plugins/management-http/src/main/java/resources/js/qpid/management/addVirtualHostNodeAndVirtualHost.js +++ b/qpid/java/broker-plugins/management-http/src/main/java/resources/js/qpid/management/addVirtualHostNodeAndVirtualHost.js @@ -27,6 +27,8 @@ define(["dojo/_base/xhr", "dojo/json", "dojo/parser", "dojo/store/Memory", + "dojo/window", + "dojo/on", "dojox/lang/functional/object", "dijit/registry", "dijit/Dialog", @@ -45,7 +47,7 @@ define(["dojo/_base/xhr", "dojox/validate/us", "dojox/validate/web", "dojo/domReady!"], - function (xhr, event, lang, array, dom, domConstruct, json, parser, Memory, fobject, registry, Dialog, Button, FilteringSelect, properties, util, metadata, template) + function (xhr, event, lang, array, dom, domConstruct, json, parser, Memory, win, on, fobject, registry, Dialog, Button, FilteringSelect, properties, util, metadata, template) { var addVirtualHostNodeAndVirtualHost = @@ -94,7 +96,6 @@ define(["dojo/_base/xhr", this.virtualHostType.set("store", this.virtualHostTypeStore); this.virtualHostType.set("disabled", false); this.virtualHostType.on("change", function(type){that._vhTypeChanged(type, that.virtualHostTypeFieldsContainer, "qpid/management/virtualhost/");}); - }, show: function() { @@ -163,6 +164,11 @@ define(["dojo/_base/xhr", this.virtualHostNodeContext.setData({},effectiveValues,inheritedActualValues); this.virtualHostContext.setData({},effectiveValues,inheritedActualValues); this.dialog.show(); + if (!this.resizeEventRegistered) + { + this.resizeEventRegistered = true; + util.resizeContentAreaAndRepositionDialog(dom.byId("addVirtualHostNodeAndVirtualHost.contentPane"), this.dialog); + } }, destroy: function() { diff --git a/qpid/java/broker-plugins/management-http/src/main/java/resources/js/qpid/management/editBroker.js b/qpid/java/broker-plugins/management-http/src/main/java/resources/js/qpid/management/editBroker.js index d83db74b65..07938fe8e7 100644 --- a/qpid/java/broker-plugins/management-http/src/main/java/resources/js/qpid/management/editBroker.js +++ b/qpid/java/broker-plugins/management-http/src/main/java/resources/js/qpid/management/editBroker.js @@ -68,8 +68,9 @@ define(["dojo/_base/xhr", this[fieldName] = registry.byId("editBroker." + fieldName); } this.form = registry.byId("editBrokerForm"); + this.form.on("submit", function(){return false;}); this.context = registry.byId("editBroker.context"); - util.applyMetadataToWidgets(this.containerNode, "Broker", "broker"); + util.applyMetadataToWidgets(dom.byId("editBroker.allFields"), "Broker", "broker"); for(var i = 0; i < numericFieldNames.length; i++) { @@ -186,6 +187,11 @@ define(["dojo/_base/xhr", this.context.load(this.query, {actualValues: actualData.context, effectiveValues: effectiveData.context}); this.dialog.startup(); this.dialog.show(); + if (!this.resizeEventRegistered) + { + this.resizeEventRegistered = true; + util.resizeContentAreaAndRepositionDialog(dom.byId("editBroker.contentPane"), this.dialog); + } } }; diff --git a/qpid/java/broker-plugins/management-http/src/main/java/resources/js/qpid/management/editVirtualHost.js b/qpid/java/broker-plugins/management-http/src/main/java/resources/js/qpid/management/editVirtualHost.js index 7ba86e5d03..5fca2d5523 100644 --- a/qpid/java/broker-plugins/management-http/src/main/java/resources/js/qpid/management/editVirtualHost.js +++ b/qpid/java/broker-plugins/management-http/src/main/java/resources/js/qpid/management/editVirtualHost.js @@ -70,6 +70,7 @@ define(["dojo/_base/xhr", this[fieldName] = registry.byId("editVirtualHost." + fieldName); } this.form = registry.byId("editVirtualHostForm"); + this.form.on("submit", function(){return false;}); }, show: function(hostData) { @@ -204,6 +205,11 @@ define(["dojo/_base/xhr", this.dialog.startup(); this.dialog.show(); + if (!this.resizeEventRegistered) + { + this.resizeEventRegistered = true; + util.resizeContentAreaAndRepositionDialog(dom.byId("editVirtualHost.contentPane"), this.dialog); + } } }; diff --git a/qpid/java/broker-plugins/management-http/src/main/java/resources/js/qpid/management/editVirtualHostNode.js b/qpid/java/broker-plugins/management-http/src/main/java/resources/js/qpid/management/editVirtualHostNode.js index 1c7cbbe439..4f815f1f1d 100644 --- a/qpid/java/broker-plugins/management-http/src/main/java/resources/js/qpid/management/editVirtualHostNode.js +++ b/qpid/java/broker-plugins/management-http/src/main/java/resources/js/qpid/management/editVirtualHostNode.js @@ -61,6 +61,7 @@ define(["dojo/_base/xhr", this.saveButton.on("click", function(e){that._save(e);}); this.name = registry.byId("editVirtualHostNode.name"); this.form = registry.byId("editVirtualHostNodeForm"); + this.form.on("submit", function(){return false;}); }, show: function(effectiveData) { @@ -173,6 +174,11 @@ define(["dojo/_base/xhr", this.dialog.startup(); this.dialog.show(); + if (!this.resizeEventRegistered) + { + this.resizeEventRegistered = true; + util.resizeContentAreaAndRepositionDialog(dom.byId("editVirtualHostNode.contentPane"), this.dialog); + } } }; -- cgit v1.2.1