Skip to content

Commit cce5107

Browse files
committed
Fixed #961
1 parent bd4a28a commit cce5107

File tree

2 files changed

+7
-3
lines changed

2 files changed

+7
-3
lines changed

gwt-material/src/main/java/gwt/material/design/client/base/mixin/RequiredFieldMixin.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@
3232
public class RequiredFieldMixin <T extends AbstractValueWidget & HasRequiredField, H extends UIObject>
3333
extends AbstractMixin<T> implements HasRequiredField {
3434

35-
private String REQUIRED = "required";
35+
private final String REQUIRED = "required";
3636
private H targetLabel;
3737
private boolean required;
3838
private ToggleStyleMixin<UIObject> toggleStyleMixin;
@@ -51,8 +51,8 @@ public RequiredFieldMixin(T uiObject, H targetLabel) {
5151
public void setRequired(boolean required) {
5252
this.required = required;
5353

54-
uiObject.setValidateOnBlur(true);
55-
uiObject.setAllowBlank(false);
54+
uiObject.setValidateOnBlur(required);
55+
uiObject.setAllowBlank(!required);
5656

5757
if (targetLabel != null) {
5858
getToggleStyleMixin().setOn(required);

gwt-material/src/test/java/gwt/material/design/client/ui/MaterialValueBoxTest.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -112,10 +112,14 @@ protected <W extends MaterialValueBox> void checkMandatoryField(W widget) {
112112
widget.setRequired(true);
113113
assertTrue(widget.isRequired());
114114
assertTrue(widget.getStatusTextMixin().getPlaceholder().getElement().hasClassName(REQUIRED));
115+
assertTrue(widget.isValidateOnBlur());
116+
assertFalse(widget.isAllowBlank());
115117

116118
widget.setRequired(false);
117119
assertFalse(widget.isRequired());
118120
assertFalse(widget.getStatusTextMixin().getPlaceholder().getElement().hasClassName(REQUIRED));
121+
assertFalse(widget.isValidateOnBlur());
122+
assertTrue(widget.isAllowBlank());
119123
}
120124

121125
protected <W extends MaterialValueBox> void checkAutocomplete(W widget) {

0 commit comments

Comments
 (0)