7:38 a.m.
Changes:
1) Have the "Manage Media" function deleted from "Action" list
2) Enable "Edit" whether VM is running or off
3) Disable edit function of "General", "Storage",
"Interface" except
"replace" of the cdrom under "Storage".
Wen Wang (2):
UI: Delete Manage Media function from action list
Allow admin user change permission settings when VM is running
ui/js/src/kimchi.guest_edit_main.js | 54 ++++++++++++++++++++++----------
ui/js/src/kimchi.guest_main.js | 14 --------
ui/pages/guest-edit.html.tmpl | 2 +-
ui/pages/guest-media.html.tmpl | 57 -----------------------------------
ui/pages/guest.html.tmpl | 1 -
5 files changed, 38 insertions(+), 90 deletions(-)
delete mode 100644 ui/pages/guest-media.html.tmpl
7:38 a.m.
New subject: [PATCH 1/2] UI: Delete Manage Media function from action list
Have the "Manage Media" function deleted from "Action" list
Signed-off-by: Wen Wang <wenwang(a)linux.vnet.ibm.com>
---
ui/js/src/kimchi.guest_main.js | 13 ---------
ui/pages/guest-media.html.tmpl | 57 ----------------------------------------
ui/pages/guest.html.tmpl | 1 -
3 files changed, 0 insertions(+), 71 deletions(-)
delete mode 100644 ui/pages/guest-media.html.tmpl
diff --git a/ui/js/src/kimchi.guest_main.js b/ui/js/src/kimchi.guest_main.js
index ff6f2e1..320847e 100644
--- a/ui/js/src/kimchi.guest_main.js
+++ b/ui/js/src/kimchi.guest_main.js
@@ -123,18 +123,6 @@ kimchi.vmdelete = function(event) {
});
};
-kimchi.vmmedia = function(event) {
- var vm = $(this).closest('li[name=guest]');
- var vm_id = vm.attr('id');
- kimchi.selectedGuest = vm_id;
- kimchi.window.open({
- url: 'guest-media.html',
- close: function() {
- kimchi.clearGuestMedia();
- }
- });
-};
-
kimchi.vmedit = function(event) {
var vm = $(this).closest('li[name=guest]');
var vm_id=vm.attr("id");
@@ -293,7 +281,6 @@ kimchi.createGuestLi = function(vmObject, prevScreenImage, openMenu)
{
if (vmRunningBool) { //If the guest is not running, do not enable shutdown
guestActions.find("[name=vm-shutdown]").on({click :
kimchi.vmshutdown});
}
- guestActions.find("[name=vm-media]").on({click : kimchi.vmmedia});
guestActions.find("[name=vm-edit]").on({click : kimchi.vmedit});
guestActions.find("[name=vm-delete]").on({click : kimchi.vmdelete});
diff --git a/ui/pages/guest-media.html.tmpl b/ui/pages/guest-media.html.tmpl
deleted file mode 100644
index 5435f52..0000000
--- a/ui/pages/guest-media.html.tmpl
+++ /dev/null
@@ -1,57 +0,0 @@
-#*
- * Project Kimchi
- *
- * Copyright IBM, Corp. 2014
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- *#
-#unicode UTF-8
-#import gettext
-#silent t = gettext.translation($lang.domain, $lang.localedir, languages=$lang.lang)
-#silent _ = t.gettext
-#silent _t = t.gettext
-
-<div id="guest-media-window" class="window">
- <header>
- <h1 class="title">$_("Manage Media")</h1>
- <div class="close">X</div>
- </header>
- <div class="content">
- <form id="form-guest-media">
- <fieldset
- id="guest-edit-cdrom-row-container"
- class="guest-edit-fieldset guest-edit-cdrom-row-container">
- </fieldset>
- </form>
- </div>
-</div>
-<script id="cdrom-row-tmpl" type="text/html">
- <div>
- <div class="guest-edit-wrapper-label">
- <label for="cdrom-{dev}">{dev}</label>
- </div>
- <div class="guest-edit-wrapper-controls">
- <input id="cdrom-{dev}" name="cdrom"
type="text"
- data-vm="{vm}" data-dev="{dev}"
- value="{path}" readonly="readonly" />
- <button class="guest-edit-cdrom-button replace"
- data-vm="{vm}" data-dev="{dev}"
- title="$_("Replace")">
- </button>
- </div>
- </div>
-</script>
-
-<script type="text/javascript">
- kimchi.guest_media_main();
-</script>
diff --git a/ui/pages/guest.html.tmpl b/ui/pages/guest.html.tmpl
index d67ef3e..43fb350 100644
--- a/ui/pages/guest.html.tmpl
+++ b/ui/pages/guest.html.tmpl
@@ -56,7 +56,6 @@
<span
class="text">$_("Actions")</span><span
class="arrow"></span>
<div class="popover actionsheet right-side"
style="width: 250px">
<button class="button-big shutoff-disabled"
name="vm-console" ><span
class="text">$_("Connect")</span></button>
- <button class="button-big shutoff-disabled"
name="vm-media"><span class="text">$_("Manage
Media")</span></button>
<button class="button-big running-disabled"
name="vm-edit"><span
class="text">$_("Edit")</span></button>
<button class="button-big shutoff-hidden"
name="vm-reset"><span
class="text">$_("Reset")</span></button>
<button class="button-big shutoff-hidden"
name="vm-shutdown"><span class="text">$_("Shut
Down")</span></button>
--
1.7.1
10:43 a.m.
New subject: [PATCH 1/2] UI: Delete Manage Media function from action list
On 21-08-2014 08:38, Wen Wang wrote:
Have the "Manage Media" function deleted from
"Action" list
Signed-off-by: Wen Wang <wenwang(a)linux.vnet.ibm.com>
The file ui/css/theme-default/guest-media.css doesn't seem to be used
anywhere else after removing those files. Please remove it as well.
9:20 p.m.
New subject: [PATCH 1/2] UI: Delete Manage Media function from action list
ACK
Thanks
On 08/21/2014 10:43 PM, CrÃstian Viana wrote:
On 21-08-2014 08:38, Wen Wang wrote:
> Have the "Manage Media" function deleted from "Action" list
>
> Signed-off-by: Wen Wang <wenwang(a)linux.vnet.ibm.com>
>
The file ui/css/theme-default/guest-media.css doesn't seem to be used
anywhere else after removing those files. Please remove it as well.
7:38 a.m.
New subject: [PATCH 2/2] Allow admin user change permission settings when VM is running
Functions changed:
1) Enable "Edit" whether VM is running or off
2) Disable edit function of "General", "Storage",
"Interface" except
"replace" of the cdrom under "Storage".
Signed-off-by: Wen Wang <wenwang(a)linux.vnet.ibm.com>
---
ui/js/src/kimchi.guest_edit_main.js | 54 ++++++++++++++++++++++++-----------
ui/js/src/kimchi.guest_main.js | 1 -
ui/pages/guest-edit.html.tmpl | 2 +-
3 files changed, 38 insertions(+), 19 deletions(-)
diff --git a/ui/js/src/kimchi.guest_edit_main.js b/ui/js/src/kimchi.guest_edit_main.js
index 01d8045..ab63bb2 100644
--- a/ui/js/src/kimchi.guest_edit_main.js
+++ b/ui/js/src/kimchi.guest_edit_main.js
@@ -19,7 +19,12 @@ kimchi.guest_edit_main = function() {
var buttonContainer = $('#action-button-container');
$('#guest-edit-tabs').tabs({
beforeActivate: function(event, ui) {
- var display_list = ['form-guest-edit-general',
'form-guest-edit-permission']
+ var display_list = null;
+ if(kimchi.thisVMState === "running") {
+ display_list = ['form-guest-edit-permission'];
+ } else {
+ display_list = ['form-guest-edit-general',
'form-guest-edit-permission'];
+ }
$(buttonContainer).addClass('hidden');
var deactivated = ui['newPanel'];
if(display_list.indexOf($(deactivated).attr('id')) >= 0) {
@@ -52,12 +57,17 @@ kimchi.guest_edit_main = function() {
text: false
});
- $('.detach', container).button({
- icons: {
- primary: 'ui-icon-trash'
- },
- text: false
- });
+ if(kimchi.thisVMState != "running") {
+ $('.detach', container).button({
+ icons: {
+ primary: 'ui-icon-trash'
+ },
+ text: false
+ });
+
+ } else {
+ $('.detach', container).remove();
+ }
$('.save', container).button({
icons: {
@@ -170,6 +180,9 @@ kimchi.guest_edit_main = function() {
var itemNode =
$.parseHTML(kimchi.substitute($('#interface-tmpl').html(),data));
$(".body",
"#form-guest-edit-interface").append(itemNode);
$("select", itemNode).append(networkOptions);
+ if(kimchi.thisVMState === "running") {
+ $("#form-guest-edit-interface .delete").remove();
+ }
if(data.network!==""){
$("select", itemNode).val(data.network);
}
@@ -340,18 +353,25 @@ kimchi.guest_edit_main = function() {
var initContent = function(guest) {
guest['icon'] = guest['icon'] || 'images/icon-vm.png';
$('#form-guest-edit-general').fillWithObject(guest);
+ kimchi.thisVMState = guest['state'];
refreshCDROMs();
-
- $('#guest-edit-attach-cdrom-button').button({
- icons: {
- primary: "ui-icon-plusthick"
- },
- text: false
- }).click(function(event) {
- event.preventDefault();
- kimchi.window.open("guest-storage-add.html");
- });
+ if(kimchi.thisVMState === "running") {
+ $("#form-guest-edit-general input").prop("disabled",
"disabled");
+ $("#guest-edit-attach-cdrom-button").remove();
+ $("#form-guest-edit-interface .header button").remove();
+ } else {
+ $('#guest-edit-attach-cdrom-button').button({
+ icons: {
+ primary: "ui-icon-plusthick"
+ },
+ text: false
+ }).click(function(event) {
+ event.preventDefault();
+ kimchi.window.open("guest-storage-add.html");
+ });
+ $("#action-button-container").removeClass("hidden");
+ }
var onAttached = function(params) {
refreshCDROMs();
diff --git a/ui/js/src/kimchi.guest_main.js b/ui/js/src/kimchi.guest_main.js
index 320847e..5cbdb40 100644
--- a/ui/js/src/kimchi.guest_main.js
+++ b/ui/js/src/kimchi.guest_main.js
@@ -253,7 +253,6 @@ kimchi.createGuestLi = function(vmObject, prevScreenImage, openMenu)
{
//Setup the VM Actions
var guestActions=result.find("div[name=guest-actions]");
guestActions.find(".shutoff-disabled").prop('disabled',
!vmRunningBool );
- guestActions.find(".running-disabled").prop('disabled',
vmRunningBool );
if (vmRunningBool) {
guestActions.find(".running-hidden").hide();
diff --git a/ui/pages/guest-edit.html.tmpl b/ui/pages/guest-edit.html.tmpl
index ed7ddeb..47615be 100644
--- a/ui/pages/guest-edit.html.tmpl
+++ b/ui/pages/guest-edit.html.tmpl
@@ -145,7 +145,7 @@
</div>
</div>
<footer>
- <div id="action-button-container" class="btn-group">
+ <div id="action-button-container" class="btn-group
hidden">
<button id="guest-edit-button-save"
class="btn-normal">
<span class="text">$_("Save")</span>
</button>
--
1.7.1
2:30 p.m.
New subject: [PATCH 2/2] Allow admin user change permission settings when VM is running
Some comments:
1) When VM is running, the edit dialog blinks when it is open to fill
the input boxes. (ie, in a first moment the input boxes are displayed
empty and after some seconds it is filled)
From my understanding it should not happen as the data will be
always displayed, the difference is if it is enabled or disabled for
edition.
Could you check it and maybe do some improvements?
2) There is a different behavior in Storage tab to Network tab.
In the Storage tab the disks (that can not be edited) are listed
without any icon BUT in the Network tab, there is an edit icon disabled.
I suggest to remove the icon when edition is not allowed.
On 08/21/2014 08:38 AM, Wen Wang wrote:
Functions changed:
1) Enable "Edit" whether VM is running or off
2) Disable edit function of "General", "Storage",
"Interface" except
"replace" of the cdrom under "Storage".
Signed-off-by: Wen Wang <wenwang(a)linux.vnet.ibm.com>
---
ui/js/src/kimchi.guest_edit_main.js | 54 ++++++++++++++++++++++++-----------
ui/js/src/kimchi.guest_main.js | 1 -
ui/pages/guest-edit.html.tmpl | 2 +-
3 files changed, 38 insertions(+), 19 deletions(-)
diff --git a/ui/js/src/kimchi.guest_edit_main.js b/ui/js/src/kimchi.guest_edit_main.js
index 01d8045..ab63bb2 100644
--- a/ui/js/src/kimchi.guest_edit_main.js
+++ b/ui/js/src/kimchi.guest_edit_main.js
@@ -19,7 +19,12 @@ kimchi.guest_edit_main = function() {
var buttonContainer = $('#action-button-container');
$('#guest-edit-tabs').tabs({
beforeActivate: function(event, ui) {
- var display_list = ['form-guest-edit-general',
'form-guest-edit-permission']
+ var display_list = null;
+ if(kimchi.thisVMState === "running") {
+ display_list = ['form-guest-edit-permission'];
+ } else {
+ display_list = ['form-guest-edit-general',
'form-guest-edit-permission'];
+ }
$(buttonContainer).addClass('hidden');
var deactivated = ui['newPanel'];
if(display_list.indexOf($(deactivated).attr('id')) >= 0) {
@@ -52,12 +57,17 @@ kimchi.guest_edit_main = function() {
text: false
});
- $('.detach', container).button({
- icons: {
- primary: 'ui-icon-trash'
- },
- text: false
- });
+ if(kimchi.thisVMState != "running") {
+ $('.detach', container).button({
+ icons: {
+ primary: 'ui-icon-trash'
+ },
+ text: false
+ });
+
+ } else {
+ $('.detach', container).remove();
+ }
$('.save', container).button({
icons: {
@@ -170,6 +180,9 @@ kimchi.guest_edit_main = function() {
var itemNode =
$.parseHTML(kimchi.substitute($('#interface-tmpl').html(),data));
$(".body",
"#form-guest-edit-interface").append(itemNode);
$("select", itemNode).append(networkOptions);
+ if(kimchi.thisVMState === "running") {
+ $("#form-guest-edit-interface .delete").remove();
+ }
if(data.network!==""){
$("select", itemNode).val(data.network);
}
@@ -340,18 +353,25 @@ kimchi.guest_edit_main = function() {
var initContent = function(guest) {
guest['icon'] = guest['icon'] || 'images/icon-vm.png';
$('#form-guest-edit-general').fillWithObject(guest);
+ kimchi.thisVMState = guest['state'];
refreshCDROMs();
-
- $('#guest-edit-attach-cdrom-button').button({
- icons: {
- primary: "ui-icon-plusthick"
- },
- text: false
- }).click(function(event) {
- event.preventDefault();
- kimchi.window.open("guest-storage-add.html");
- });
+ if(kimchi.thisVMState === "running") {
+ $("#form-guest-edit-general input").prop("disabled",
"disabled");
+ $("#guest-edit-attach-cdrom-button").remove();
+ $("#form-guest-edit-interface .header button").remove();
+ } else {
+ $('#guest-edit-attach-cdrom-button').button({
+ icons: {
+ primary: "ui-icon-plusthick"
+ },
+ text: false
+ }).click(function(event) {
+ event.preventDefault();
+ kimchi.window.open("guest-storage-add.html");
+ });
+ $("#action-button-container").removeClass("hidden");
+ }
var onAttached = function(params) {
refreshCDROMs();
diff --git a/ui/js/src/kimchi.guest_main.js b/ui/js/src/kimchi.guest_main.js
index 320847e..5cbdb40 100644
--- a/ui/js/src/kimchi.guest_main.js
+++ b/ui/js/src/kimchi.guest_main.js
@@ -253,7 +253,6 @@ kimchi.createGuestLi = function(vmObject, prevScreenImage, openMenu)
{
//Setup the VM Actions
var guestActions=result.find("div[name=guest-actions]");
guestActions.find(".shutoff-disabled").prop('disabled',
!vmRunningBool );
- guestActions.find(".running-disabled").prop('disabled',
vmRunningBool );
if (vmRunningBool) {
guestActions.find(".running-hidden").hide();
diff --git a/ui/pages/guest-edit.html.tmpl b/ui/pages/guest-edit.html.tmpl
index ed7ddeb..47615be 100644
--- a/ui/pages/guest-edit.html.tmpl
+++ b/ui/pages/guest-edit.html.tmpl
@@ -145,7 +145,7 @@
</div>
</div>
<footer>
- <div id="action-button-container" class="btn-group">
+ <div id="action-button-container" class="btn-group
hidden">
<button id="guest-edit-button-save"
class="btn-normal">
<span class="text">$_("Save")</span>
</button>
9:21 p.m.
New subject: [PATCH 2/2] Allow admin user change permission settings when VM is running
Just some inline comments. Thanks Aline.
On 08/22/2014 02:30 AM, Aline Manera wrote:
Some comments:
1) When VM is running, the edit dialog blinks when it is open to fill
the input boxes. (ie, in a first moment the input boxes are displayed
empty and after some seconds it is filled)
From my understanding it should not happen as the data will be
always displayed, the difference is if it is enabled or disabled for
edition.
Could you check it and maybe do some improvements?
I will look into it
2) There is a different behavior in Storage tab to Network tab.
In the Storage tab the disks (that can not be edited) are listed
without any icon BUT in the Network tab, there is an edit icon disabled.
I suggest to remove the icon when edition is not allowed.
ACK
On 08/21/2014 08:38 AM, Wen Wang wrote:
> Functions changed:
> 1) Enable "Edit" whether VM is running or off
> 2) Disable edit function of "General", "Storage",
"Interface" except
> "replace" of the cdrom under "Storage".
>
> Signed-off-by: Wen Wang <wenwang(a)linux.vnet.ibm.com>
> ---
> ui/js/src/kimchi.guest_edit_main.js | 54
> ++++++++++++++++++++++++-----------
> ui/js/src/kimchi.guest_main.js | 1 -
> ui/pages/guest-edit.html.tmpl | 2 +-
> 3 files changed, 38 insertions(+), 19 deletions(-)
>
> diff --git a/ui/js/src/kimchi.guest_edit_main.js
> b/ui/js/src/kimchi.guest_edit_main.js
> index 01d8045..ab63bb2 100644
> --- a/ui/js/src/kimchi.guest_edit_main.js
> +++ b/ui/js/src/kimchi.guest_edit_main.js
> @@ -19,7 +19,12 @@ kimchi.guest_edit_main = function() {
> var buttonContainer = $('#action-button-container');
> $('#guest-edit-tabs').tabs({
> beforeActivate: function(event, ui) {
> - var display_list = ['form-guest-edit-general',
> 'form-guest-edit-permission']
> + var display_list = null;
> + if(kimchi.thisVMState === "running") {
> + display_list = ['form-guest-edit-permission'];
> + } else {
> + display_list = ['form-guest-edit-general',
> 'form-guest-edit-permission'];
> + }
> $(buttonContainer).addClass('hidden');
> var deactivated = ui['newPanel'];
> if(display_list.indexOf($(deactivated).attr('id')) >= 0) {
> @@ -52,12 +57,17 @@ kimchi.guest_edit_main = function() {
> text: false
> });
>
> - $('.detach', container).button({
> - icons: {
> - primary: 'ui-icon-trash'
> - },
> - text: false
> - });
> + if(kimchi.thisVMState != "running") {
> + $('.detach', container).button({
> + icons: {
> + primary: 'ui-icon-trash'
> + },
> + text: false
> + });
> +
> + } else {
> + $('.detach', container).remove();
> + }
>
> $('.save', container).button({
> icons: {
> @@ -170,6 +180,9 @@ kimchi.guest_edit_main = function() {
> var itemNode =
> $.parseHTML(kimchi.substitute($('#interface-tmpl').html(),data));
> $(".body",
"#form-guest-edit-interface").append(itemNode);
> $("select", itemNode).append(networkOptions);
> + if(kimchi.thisVMState === "running") {
> + $("#form-guest-edit-interface .delete").remove();
> + }
> if(data.network!==""){
> $("select", itemNode).val(data.network);
> }
> @@ -340,18 +353,25 @@ kimchi.guest_edit_main = function() {
> var initContent = function(guest) {
> guest['icon'] = guest['icon'] ||
'images/icon-vm.png';
> $('#form-guest-edit-general').fillWithObject(guest);
> + kimchi.thisVMState = guest['state'];
>
> refreshCDROMs();
> -
> - $('#guest-edit-attach-cdrom-button').button({
> - icons: {
> - primary: "ui-icon-plusthick"
> - },
> - text: false
> - }).click(function(event) {
> - event.preventDefault();
> - kimchi.window.open("guest-storage-add.html");
> - });
> + if(kimchi.thisVMState === "running") {
> + $("#form-guest-edit-general input").prop("disabled",
> "disabled");
> + $("#guest-edit-attach-cdrom-button").remove();
> + $("#form-guest-edit-interface .header button").remove();
> + } else {
> + $('#guest-edit-attach-cdrom-button').button({
> + icons: {
> + primary: "ui-icon-plusthick"
> + },
> + text: false
> + }).click(function(event) {
> + event.preventDefault();
> + kimchi.window.open("guest-storage-add.html");
> + });
> + $("#action-button-container").removeClass("hidden");
> + }
>
> var onAttached = function(params) {
> refreshCDROMs();
> diff --git a/ui/js/src/kimchi.guest_main.js
> b/ui/js/src/kimchi.guest_main.js
> index 320847e..5cbdb40 100644
> --- a/ui/js/src/kimchi.guest_main.js
> +++ b/ui/js/src/kimchi.guest_main.js
> @@ -253,7 +253,6 @@ kimchi.createGuestLi = function(vmObject,
> prevScreenImage, openMenu) {
> //Setup the VM Actions
> var guestActions=result.find("div[name=guest-actions]");
> guestActions.find(".shutoff-disabled").prop('disabled',
> !vmRunningBool );
> - guestActions.find(".running-disabled").prop('disabled',
> vmRunningBool );
>
> if (vmRunningBool) {
> guestActions.find(".running-hidden").hide();
> diff --git a/ui/pages/guest-edit.html.tmpl
> b/ui/pages/guest-edit.html.tmpl
> index ed7ddeb..47615be 100644
> --- a/ui/pages/guest-edit.html.tmpl
> +++ b/ui/pages/guest-edit.html.tmpl
> @@ -145,7 +145,7 @@
> </div>
> </div>
> <footer>
> - <div id="action-button-container"
class="btn-group">
> + <div id="action-button-container" class="btn-group
hidden">
> <button id="guest-edit-button-save"
class="btn-normal">
> <span
class="text">$_("Save")</span>
> </button>
2:03 a.m.
New subject: [PATCH 2/2] Allow admin user change permission settings when VM is running
On 8/22/2014 2:30 AM, Aline Manera wrote:
Some comments:
1) When VM is running, the edit dialog blinks when it is open to fill
the input boxes. (ie, in a first moment the input boxes are displayed
empty and after some seconds it is filled)
From my understanding it should not happen as the data will be
always displayed, the difference is if it is enabled or disabled for
edition.
Could you check it and maybe do some improvements?
I have tried to rebase for
the investigation. By adding no code just
enable the "Edit" button when VM is running, the time-delay still
exists. So this should probably not be the problem that front-end cause.
Could you please help with the investigation in the back-end? Could it
be the problem caused by retrieving information delay when VM is running?
2) There is a different behavior in Storage tab to Network tab.
In the Storage tab the disks (that can not be edited) are listed
without any icon BUT in the Network tab, there is an edit icon disabled.
I suggest to remove the icon when edition is not allowed.
I have fixed this in V2
On 08/21/2014 08:38 AM, Wen Wang wrote:
> Functions changed:
> 1) Enable "Edit" whether VM is running or off
> 2) Disable edit function of "General", "Storage",
"Interface" except
> "replace" of the cdrom under "Storage".
>
> Signed-off-by: Wen Wang <wenwang(a)linux.vnet.ibm.com>
> ---
> ui/js/src/kimchi.guest_edit_main.js | 54
> ++++++++++++++++++++++++-----------
> ui/js/src/kimchi.guest_main.js | 1 -
> ui/pages/guest-edit.html.tmpl | 2 +-
> 3 files changed, 38 insertions(+), 19 deletions(-)
>
> diff --git a/ui/js/src/kimchi.guest_edit_main.js
> b/ui/js/src/kimchi.guest_edit_main.js
> index 01d8045..ab63bb2 100644
> --- a/ui/js/src/kimchi.guest_edit_main.js
> +++ b/ui/js/src/kimchi.guest_edit_main.js
> @@ -19,7 +19,12 @@ kimchi.guest_edit_main = function() {
> var buttonContainer = $('#action-button-container');
> $('#guest-edit-tabs').tabs({
> beforeActivate: function(event, ui) {
> - var display_list = ['form-guest-edit-general',
> 'form-guest-edit-permission']
> + var display_list = null;
> + if(kimchi.thisVMState === "running") {
> + display_list = ['form-guest-edit-permission'];
> + } else {
> + display_list = ['form-guest-edit-general',
> 'form-guest-edit-permission'];
> + }
> $(buttonContainer).addClass('hidden');
> var deactivated = ui['newPanel'];
> if(display_list.indexOf($(deactivated).attr('id')) >= 0) {
> @@ -52,12 +57,17 @@ kimchi.guest_edit_main = function() {
> text: false
> });
>
> - $('.detach', container).button({
> - icons: {
> - primary: 'ui-icon-trash'
> - },
> - text: false
> - });
> + if(kimchi.thisVMState != "running") {
> + $('.detach', container).button({
> + icons: {
> + primary: 'ui-icon-trash'
> + },
> + text: false
> + });
> +
> + } else {
> + $('.detach', container).remove();
> + }
>
> $('.save', container).button({
> icons: {
> @@ -170,6 +180,9 @@ kimchi.guest_edit_main = function() {
> var itemNode =
> $.parseHTML(kimchi.substitute($('#interface-tmpl').html(),data));
> $(".body",
"#form-guest-edit-interface").append(itemNode);
> $("select", itemNode).append(networkOptions);
> + if(kimchi.thisVMState === "running") {
> + $("#form-guest-edit-interface .delete").remove();
> + }
> if(data.network!==""){
> $("select", itemNode).val(data.network);
> }
> @@ -340,18 +353,25 @@ kimchi.guest_edit_main = function() {
> var initContent = function(guest) {
> guest['icon'] = guest['icon'] ||
'images/icon-vm.png';
> $('#form-guest-edit-general').fillWithObject(guest);
> + kimchi.thisVMState = guest['state'];
>
> refreshCDROMs();
> -
> - $('#guest-edit-attach-cdrom-button').button({
> - icons: {
> - primary: "ui-icon-plusthick"
> - },
> - text: false
> - }).click(function(event) {
> - event.preventDefault();
> - kimchi.window.open("guest-storage-add.html");
> - });
> + if(kimchi.thisVMState === "running") {
> + $("#form-guest-edit-general input").prop("disabled",
> "disabled");
> + $("#guest-edit-attach-cdrom-button").remove();
> + $("#form-guest-edit-interface .header button").remove();
> + } else {
> + $('#guest-edit-attach-cdrom-button').button({
> + icons: {
> + primary: "ui-icon-plusthick"
> + },
> + text: false
> + }).click(function(event) {
> + event.preventDefault();
> + kimchi.window.open("guest-storage-add.html");
> + });
> + $("#action-button-container").removeClass("hidden");
> + }
>
> var onAttached = function(params) {
> refreshCDROMs();
> diff --git a/ui/js/src/kimchi.guest_main.js
> b/ui/js/src/kimchi.guest_main.js
> index 320847e..5cbdb40 100644
> --- a/ui/js/src/kimchi.guest_main.js
> +++ b/ui/js/src/kimchi.guest_main.js
> @@ -253,7 +253,6 @@ kimchi.createGuestLi = function(vmObject,
> prevScreenImage, openMenu) {
> //Setup the VM Actions
> var guestActions=result.find("div[name=guest-actions]");
> guestActions.find(".shutoff-disabled").prop('disabled',
> !vmRunningBool );
> - guestActions.find(".running-disabled").prop('disabled',
> vmRunningBool );
>
> if (vmRunningBool) {
> guestActions.find(".running-hidden").hide();
> diff --git a/ui/pages/guest-edit.html.tmpl
> b/ui/pages/guest-edit.html.tmpl
> index ed7ddeb..47615be 100644
> --- a/ui/pages/guest-edit.html.tmpl
> +++ b/ui/pages/guest-edit.html.tmpl
> @@ -145,7 +145,7 @@
> </div>
> </div>
> <footer>
> - <div id="action-button-container"
class="btn-group">
> + <div id="action-button-container" class="btn-group
hidden">
> <button id="guest-edit-button-save"
class="btn-normal">
> <span
class="text">$_("Save")</span>
> </button>
10:23 a.m.
New subject: [PATCH 2/2] Allow admin user change permission settings when VM is running
On 08/22/2014 03:03 AM, Wen Wang wrote:
On 8/22/2014 2:30 AM, Aline Manera wrote:
>
> Some comments:
>
> 1) When VM is running, the edit dialog blinks when it is open to fill
> the input boxes. (ie, in a first moment the input boxes are displayed
> empty and after some seconds it is filled)
> From my understanding it should not happen as the data will be
> always displayed, the difference is if it is enabled or disabled for
> edition.
> Could you check it and maybe do some improvements?
I have tried to rebase for the investigation. By adding no code just
enable the "Edit" button when VM is running, the time-delay still
exists. So this should probably not be the problem that front-end
cause. Could you please help with the investigation in the back-end?
Could it be the problem caused by retrieving information delay when VM
is running?
Yeap. Seems the usual time responding for requests.
We can discuss how to handle it better for next release.
>
> 2) There is a different behavior in Storage tab to Network tab.
> In the Storage tab the disks (that can not be edited) are listed
> without any icon BUT in the Network tab, there is an edit icon disabled.
> I suggest to remove the icon when edition is not allowed.
>
I have fixed this in V2
>
> On 08/21/2014 08:38 AM, Wen Wang wrote:
>> Functions changed:
>> 1) Enable "Edit" whether VM is running or off
>> 2) Disable edit function of "General", "Storage",
"Interface" except
>> "replace" of the cdrom under "Storage".
>>
>> Signed-off-by: Wen Wang <wenwang(a)linux.vnet.ibm.com>
>> ---
>> ui/js/src/kimchi.guest_edit_main.js | 54
>> ++++++++++++++++++++++++-----------
>> ui/js/src/kimchi.guest_main.js | 1 -
>> ui/pages/guest-edit.html.tmpl | 2 +-
>> 3 files changed, 38 insertions(+), 19 deletions(-)
>>
>> diff --git a/ui/js/src/kimchi.guest_edit_main.js
>> b/ui/js/src/kimchi.guest_edit_main.js
>> index 01d8045..ab63bb2 100644
>> --- a/ui/js/src/kimchi.guest_edit_main.js
>> +++ b/ui/js/src/kimchi.guest_edit_main.js
>> @@ -19,7 +19,12 @@ kimchi.guest_edit_main = function() {
>> var buttonContainer = $('#action-button-container');
>> $('#guest-edit-tabs').tabs({
>> beforeActivate: function(event, ui) {
>> - var display_list = ['form-guest-edit-general',
>> 'form-guest-edit-permission']
>> + var display_list = null;
>> + if(kimchi.thisVMState === "running") {
>> + display_list = ['form-guest-edit-permission'];
>> + } else {
>> + display_list = ['form-guest-edit-general',
>> 'form-guest-edit-permission'];
>> + }
>> $(buttonContainer).addClass('hidden');
>> var deactivated = ui['newPanel'];
>> if(display_list.indexOf($(deactivated).attr('id')) >= 0) {
>> @@ -52,12 +57,17 @@ kimchi.guest_edit_main = function() {
>> text: false
>> });
>>
>> - $('.detach', container).button({
>> - icons: {
>> - primary: 'ui-icon-trash'
>> - },
>> - text: false
>> - });
>> + if(kimchi.thisVMState != "running") {
>> + $('.detach', container).button({
>> + icons: {
>> + primary: 'ui-icon-trash'
>> + },
>> + text: false
>> + });
>> +
>> + } else {
>> + $('.detach', container).remove();
>> + }
>>
>> $('.save', container).button({
>> icons: {
>> @@ -170,6 +180,9 @@ kimchi.guest_edit_main = function() {
>> var itemNode =
>> $.parseHTML(kimchi.substitute($('#interface-tmpl').html(),data));
>> $(".body",
>> "#form-guest-edit-interface").append(itemNode);
>> $("select", itemNode).append(networkOptions);
>> + if(kimchi.thisVMState === "running") {
>> + $("#form-guest-edit-interface .delete").remove();
>> + }
>> if(data.network!==""){
>> $("select", itemNode).val(data.network);
>> }
>> @@ -340,18 +353,25 @@ kimchi.guest_edit_main = function() {
>> var initContent = function(guest) {
>> guest['icon'] = guest['icon'] ||
'images/icon-vm.png';
>> $('#form-guest-edit-general').fillWithObject(guest);
>> + kimchi.thisVMState = guest['state'];
>>
>> refreshCDROMs();
>> -
>> - $('#guest-edit-attach-cdrom-button').button({
>> - icons: {
>> - primary: "ui-icon-plusthick"
>> - },
>> - text: false
>> - }).click(function(event) {
>> - event.preventDefault();
>> - kimchi.window.open("guest-storage-add.html");
>> - });
>> + if(kimchi.thisVMState === "running") {
>> + $("#form-guest-edit-general
input").prop("disabled",
>> "disabled");
>> + $("#guest-edit-attach-cdrom-button").remove();
>> + $("#form-guest-edit-interface .header button").remove();
>> + } else {
>> + $('#guest-edit-attach-cdrom-button').button({
>> + icons: {
>> + primary: "ui-icon-plusthick"
>> + },
>> + text: false
>> + }).click(function(event) {
>> + event.preventDefault();
>> + kimchi.window.open("guest-storage-add.html");
>> + });
>> + $("#action-button-container").removeClass("hidden");
>> + }
>>
>> var onAttached = function(params) {
>> refreshCDROMs();
>> diff --git a/ui/js/src/kimchi.guest_main.js
>> b/ui/js/src/kimchi.guest_main.js
>> index 320847e..5cbdb40 100644
>> --- a/ui/js/src/kimchi.guest_main.js
>> +++ b/ui/js/src/kimchi.guest_main.js
>> @@ -253,7 +253,6 @@ kimchi.createGuestLi = function(vmObject,
>> prevScreenImage, openMenu) {
>> //Setup the VM Actions
>> var guestActions=result.find("div[name=guest-actions]");
>> guestActions.find(".shutoff-disabled").prop('disabled',
>> !vmRunningBool );
>> - guestActions.find(".running-disabled").prop('disabled',
>> vmRunningBool );
>>
>> if (vmRunningBool) {
>> guestActions.find(".running-hidden").hide();
>> diff --git a/ui/pages/guest-edit.html.tmpl
>> b/ui/pages/guest-edit.html.tmpl
>> index ed7ddeb..47615be 100644
>> --- a/ui/pages/guest-edit.html.tmpl
>> +++ b/ui/pages/guest-edit.html.tmpl
>> @@ -145,7 +145,7 @@
>> </div>
>> </div>
>> <footer>
>> - <div id="action-button-container"
class="btn-group">
>> + <div id="action-button-container" class="btn-group
hidden">
>> <button id="guest-edit-button-save"
class="btn-normal">
>> <span
class="text">$_("Save")</span>
>> </button>
>
9:37 p.m.
New subject: [PATCH 2/2] Allow admin user change permission settings when VM is running
On 8/22/2014 10:23 PM, Aline Manera wrote:
On 08/22/2014 03:03 AM, Wen Wang wrote:
>
> On 8/22/2014 2:30 AM, Aline Manera wrote:
>>
>> Some comments:
>>
>> 1) When VM is running, the edit dialog blinks when it is open to
>> fill the input boxes. (ie, in a first moment the input boxes are
>> displayed empty and after some seconds it is filled)
>> From my understanding it should not happen as the data will be
>> always displayed, the difference is if it is enabled or disabled for
>> edition.
>> Could you check it and maybe do some improvements?
> I have tried to rebase for the investigation. By adding no code just
> enable the "Edit" button when VM is running, the time-delay still
> exists. So this should probably not be the problem that front-end
> cause. Could you please help with the investigation in the back-end?
> Could it be the problem caused by retrieving information delay when
> VM is running?
Yeap. Seems the usual time responding for requests.
We can discuss how to handle it better for next release.
Agreed.
>>
>> 2) There is a different behavior in Storage tab to Network tab.
>> In the Storage tab the disks (that can not be edited) are
>> listed without any icon BUT in the Network tab, there is an edit
>> icon disabled.
>> I suggest to remove the icon when edition is not allowed.
>>
> I have fixed this in V2
>>
>> On 08/21/2014 08:38 AM, Wen Wang wrote:
>>> Functions changed:
>>> 1) Enable "Edit" whether VM is running or off
>>> 2) Disable edit function of "General", "Storage",
"Interface" except
>>> "replace" of the cdrom under "Storage".
>>>
>>> Signed-off-by: Wen Wang <wenwang(a)linux.vnet.ibm.com>
>>> ---
>>> ui/js/src/kimchi.guest_edit_main.js | 54
>>> ++++++++++++++++++++++++-----------
>>> ui/js/src/kimchi.guest_main.js | 1 -
>>> ui/pages/guest-edit.html.tmpl | 2 +-
>>> 3 files changed, 38 insertions(+), 19 deletions(-)
>>>
>>> diff --git a/ui/js/src/kimchi.guest_edit_main.js
>>> b/ui/js/src/kimchi.guest_edit_main.js
>>> index 01d8045..ab63bb2 100644
>>> --- a/ui/js/src/kimchi.guest_edit_main.js
>>> +++ b/ui/js/src/kimchi.guest_edit_main.js
>>> @@ -19,7 +19,12 @@ kimchi.guest_edit_main = function() {
>>> var buttonContainer = $('#action-button-container');
>>> $('#guest-edit-tabs').tabs({
>>> beforeActivate: function(event, ui) {
>>> - var display_list = ['form-guest-edit-general',
>>> 'form-guest-edit-permission']
>>> + var display_list = null;
>>> + if(kimchi.thisVMState === "running") {
>>> + display_list = ['form-guest-edit-permission'];
>>> + } else {
>>> + display_list = ['form-guest-edit-general',
>>> 'form-guest-edit-permission'];
>>> + }
>>> $(buttonContainer).addClass('hidden');
>>> var deactivated = ui['newPanel'];
>>> if(display_list.indexOf($(deactivated).attr('id')) >= 0) {
>>> @@ -52,12 +57,17 @@ kimchi.guest_edit_main = function() {
>>> text: false
>>> });
>>>
>>> - $('.detach', container).button({
>>> - icons: {
>>> - primary: 'ui-icon-trash'
>>> - },
>>> - text: false
>>> - });
>>> + if(kimchi.thisVMState != "running") {
>>> + $('.detach', container).button({
>>> + icons: {
>>> + primary: 'ui-icon-trash'
>>> + },
>>> + text: false
>>> + });
>>> +
>>> + } else {
>>> + $('.detach', container).remove();
>>> + }
>>>
>>> $('.save', container).button({
>>> icons: {
>>> @@ -170,6 +180,9 @@ kimchi.guest_edit_main = function() {
>>> var itemNode =
>>> $.parseHTML(kimchi.substitute($('#interface-tmpl').html(),data));
>>> $(".body",
>>> "#form-guest-edit-interface").append(itemNode);
>>> $("select", itemNode).append(networkOptions);
>>> + if(kimchi.thisVMState === "running") {
>>> + $("#form-guest-edit-interface .delete").remove();
>>> + }
>>> if(data.network!==""){
>>> $("select", itemNode).val(data.network);
>>> }
>>> @@ -340,18 +353,25 @@ kimchi.guest_edit_main = function() {
>>> var initContent = function(guest) {
>>> guest['icon'] = guest['icon'] ||
'images/icon-vm.png';
>>> $('#form-guest-edit-general').fillWithObject(guest);
>>> + kimchi.thisVMState = guest['state'];
>>>
>>> refreshCDROMs();
>>> -
>>> - $('#guest-edit-attach-cdrom-button').button({
>>> - icons: {
>>> - primary: "ui-icon-plusthick"
>>> - },
>>> - text: false
>>> - }).click(function(event) {
>>> - event.preventDefault();
>>> - kimchi.window.open("guest-storage-add.html");
>>> - });
>>> + if(kimchi.thisVMState === "running") {
>>> + $("#form-guest-edit-general
input").prop("disabled",
>>> "disabled");
>>> + $("#guest-edit-attach-cdrom-button").remove();
>>> + $("#form-guest-edit-interface .header
button").remove();
>>> + } else {
>>> + $('#guest-edit-attach-cdrom-button').button({
>>> + icons: {
>>> + primary: "ui-icon-plusthick"
>>> + },
>>> + text: false
>>> + }).click(function(event) {
>>> + event.preventDefault();
>>> + kimchi.window.open("guest-storage-add.html");
>>> + });
>>> + $("#action-button-container").removeClass("hidden");
>>> + }
>>>
>>> var onAttached = function(params) {
>>> refreshCDROMs();
>>> diff --git a/ui/js/src/kimchi.guest_main.js
>>> b/ui/js/src/kimchi.guest_main.js
>>> index 320847e..5cbdb40 100644
>>> --- a/ui/js/src/kimchi.guest_main.js
>>> +++ b/ui/js/src/kimchi.guest_main.js
>>> @@ -253,7 +253,6 @@ kimchi.createGuestLi = function(vmObject,
>>> prevScreenImage, openMenu) {
>>> //Setup the VM Actions
>>> var guestActions=result.find("div[name=guest-actions]");
>>> guestActions.find(".shutoff-disabled").prop('disabled',
>>> !vmRunningBool );
>>> -
guestActions.find(".running-disabled").prop('disabled',
>>> vmRunningBool );
>>>
>>> if (vmRunningBool) {
>>> guestActions.find(".running-hidden").hide();
>>> diff --git a/ui/pages/guest-edit.html.tmpl
>>> b/ui/pages/guest-edit.html.tmpl
>>> index ed7ddeb..47615be 100644
>>> --- a/ui/pages/guest-edit.html.tmpl
>>> +++ b/ui/pages/guest-edit.html.tmpl
>>> @@ -145,7 +145,7 @@
>>> </div>
>>> </div>
>>> <footer>
>>> - <div id="action-button-container"
class="btn-group">
>>> + <div id="action-button-container" class="btn-group
hidden">
>>> <button id="guest-edit-button-save"
class="btn-normal">
>>> <span
class="text">$_("Save")</span>
>>> </button>
>>
>
>
3775
days inactive
3779
days old
10 comments
3 participants
participants (3)
-
Aline Manera -
CrÃstian Viana -
Wen Wang