[brooklyn-ui] branch master updated: fix for incorrect logic for disabling apply button

[jc...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

jcabrerizo pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/brooklyn-ui.git


The following commit(s) were added to refs/heads/master by this push:
     new 7e43e8a  fix for incorrect logic for disabling apply button
     new 3abfb35  Merge pull request #268 from zan-mateusz/fix/node-management-modal-apply-button
7e43e8a is described below

commit 7e43e8a84f84f383a00ba42902696e4c820b898e
Author: zan-mateusz <ma...@cloudsoftcorp.com>
AuthorDate: Tue Aug 17 17:16:31 2021 +0100

    fix for incorrect logic for disabling apply button
---
 .../home/app/views/about/node-management/node-management.template.html  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ui-modules/home/app/views/about/node-management/node-management.template.html b/ui-modules/home/app/views/about/node-management/node-management.template.html
index 253d1a6..5e41d53 100644
--- a/ui-modules/home/app/views/about/node-management/node-management.template.html
+++ b/ui-modules/home/app/views/about/node-management/node-management.template.html
@@ -77,7 +77,7 @@
                 ng-click="doShowEditOptions()">Edit</button>
         <button class="btn btn-lg btn-success btn-default pull-right" style="margin-left: 1em"
                 ng-if="showEditOptions"
-                ng-disabled="!vm.newPriority || (vm.newPriority === node.priority && vm.newStatus === node.status)"
+                ng-disabled="(!vm.newPriority && !vm.newStatus) || (vm.newPriority === node.priority && vm.newStatus === node.status)"
                 ng-click="applyChangesAndQuit()">Apply</button>
     </div>