Update 0.2.3 tag on npm about formsy-material-ui HOT 5 CLOSED

Version bumped in packages.json, README updated, published to npm. I didn't force reset the tag with these changes.

from formsy-material-ui.

@mbrookes I did a mistake in v0.2.3. I have fixed that mistake and applied this approach
https://github.com/christianalfoni/formsy-react/wiki/validate-on-blur
in following pr. Please have a look and let me know if I can merge.
#12

Thanks

from formsy-material-ui.

@vijayrawatsan Can you briefly explain what broke, and how this fixes it? Thanks.

from formsy-material-ui.

What broke?
Basically my previous commit added value={this.getValue() to FormsyText which was earlier ommited and looked the right thing to do(christianalfoni/formsy-react#117).
Issue was there due to onBlur,
formsy-material-ui FormsyText uses onBlur instead of onChange. Due to this after the first onBlur on any inputtext I was not able to change the value of that text field as next value change will happen on onBlur but value is referring to this.getValue()

I tried to be as clear as possible do let me know if you did not understand.

In my new pull request I used this approach to validate FormsyText
https://github.com/christianalfoni/formsy-react/wiki/validate-on-blur
which is the best possible way till now to validate and looks natural.

from formsy-material-ui.

Closing this since npm is updated (albeit with a broken version). FWIW it originally had value={this.getValue(0} when validation was onChange, but it subsequently appeared redundant. Clearly that wasn't the case! :)

We can move the discussion to the relevant PR.

from formsy-material-ui.