You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by bf...@apache.org on 2015/01/07 17:48:57 UTC

git commit: updated refs/heads/ui-template-uploader to b35fe9a

Repository: cloudstack
Updated Branches:
  refs/heads/ui-template-uploader d2a928dbf -> b35fe9ad6


UI validation: Fix asterisk showing up on non-required fields


Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/b35fe9ad
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/b35fe9ad
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/b35fe9ad

Branch: refs/heads/ui-template-uploader
Commit: b35fe9ad6686524e732abd3a0ba35ad02b11b826
Parents: d2a928d
Author: Brian Federle <br...@citrix.com>
Authored: Wed Jan 7 08:46:59 2015 -0800
Committer: Brian Federle <br...@citrix.com>
Committed: Wed Jan 7 08:46:59 2015 -0800

----------------------------------------------------------------------
 ui/scripts/ui/dialog.js | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cloudstack/blob/b35fe9ad/ui/scripts/ui/dialog.js
----------------------------------------------------------------------
diff --git a/ui/scripts/ui/dialog.js b/ui/scripts/ui/dialog.js
index 77bc3ff..ec4a417 100644
--- a/ui/scripts/ui/dialog.js
+++ b/ui/scripts/ui/dialog.js
@@ -193,11 +193,12 @@
                         );
 
                 // red asterisk
-                var $astersikSpan = $('<span>').addClass('field-required').html('*');
-                $name.find('label').prepend($astersikSpan);
+                var $asterisk = $('<span>').addClass('field-required').html('*');
 
-                if (field.validation == null || field.validation.required == false) {
-                    $astersikSpan.hide();
+                $name.find('label').prepend($asterisk);
+
+                if (field.validation == null || !field.validation.required) {
+                    $asterisk.hide();
                 }
 
                 // Tooltip description