Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

restore value if it is set and track changes using onkeyup #43

Merged

Conversation

mikey0000
Copy link

the only issue here is the same as I brought up before in #39 .

If the value is the numerical 0 it will erase the value

@kysonic kysonic merged commit dfbb8cd into kysonic:master Mar 31, 2017
Joxit added a commit to Joxit/riot-mui that referenced this pull request Apr 1, 2017
kysonic#43 was not compatible with riot v3
Close issue kysonic#42
@@ -22,6 +22,8 @@
if(opts.maxRows) this.refs.mirror.style.maxHeight = opts.maxRows*this.refs[this.n].getBoundingClientRect().height + 'px';
this.n = opts.ref||'default-textarea';
// Defaults
this.value = opts.value || '';
this.mirror.innerHTML = this.format(this.value);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Be carefull, this is not compatible with riot v3 and the latest version of riot-mui.

Please see riot v3 breaking changes

kysonic added a commit that referenced this pull request Apr 1, 2017
Fix for material-textarea (#42 and #43)
@mikey0000 mikey0000 deleted the bugfix/material-textarea-value-not-updating branch July 13, 2017 05:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants