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:49:58 UTC
git commit: updated refs/heads/master to 82fa918
Repository: cloudstack
Updated Branches:
refs/heads/master 85bd96c6b -> 82fa9184b
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/82fa9184
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/82fa9184
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/82fa9184
Branch: refs/heads/master
Commit: 82fa9184bfe5f59443a101e1008d57ce77cce53a
Parents: 85bd96c
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:49:27 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/82fa9184/ui/scripts/ui/dialog.js
----------------------------------------------------------------------
diff --git a/ui/scripts/ui/dialog.js b/ui/scripts/ui/dialog.js
index e5ac14b..051759f 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