Skip to content

add trim modifier #374

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

Merged
merged 4 commits into from
Aug 22, 2016
Merged

add trim modifier #374

merged 4 commits into from
Aug 22, 2016

Conversation

kazupon
Copy link
Member

@kazupon kazupon commented Aug 20, 2016

No description provided.


### `.trim`

If you want user input to be automatically trim, tou can add the `trim` modifier to youre `v-model` managed inputs:
Copy link
Member

@LinusBorg LinusBorg Aug 21, 2016

Choose a reason for hiding this comment

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

If you want user input to be automatically trim

Better would be:

If you want user input to be trimmed automatically...

Copy link
Member Author

Choose a reason for hiding this comment

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

Thank you for your review!
sorry, poor my english 😿

@@ -347,7 +347,7 @@ This is often useful, because even with `type="number"`, the value of HTML input

### `.trim`

If you want user input to be automatically trim, tou can add the `trim` modifier to youre `v-model` managed inputs:
If you want user input to trimmed automatically, you can add the `trim` modifier to youre `v-model` managed inputs:
Copy link
Member

Choose a reason for hiding this comment

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

Sorry to bother you with this again, but there is still a grammar error:

If you want user input to be trimmed automatically,

be is missing.

Copy link
Member Author

Choose a reason for hiding this comment

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

oops, I fixed now.

@@ -347,7 +347,7 @@ This is often useful, because even with `type="number"`, the value of HTML input

### `.trim`

If you want user input to trimmed automatically, you can add the `trim` modifier to youre `v-model` managed inputs:
If you want user input to be trimmed automatically, you can add the `trim` modifier to youre `v-model` managed inputs:
Copy link
Contributor

Choose a reason for hiding this comment

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

Just one more fix. 😃 youre should be your.

Copy link
Member Author

Choose a reason for hiding this comment

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

oooops, 😹
fixed now.

@chrisvfritz chrisvfritz merged commit 9e1697f into vuejs:2.0 Aug 22, 2016
dingyiming referenced this pull request in volksbright/document-standard-vue Aug 22, 2016
* 2.0:
  add trim modifier (#374)
  Update vue_version
  add instance methods to API docs
  add directives to API docs
  remove outdated and nonsensical example.
  Remove ElementDirective example, as this functionality was deprecated.
  slim down lists of referenced companion libs
  update one more function component slots syntax example
  update slots syntax for render function
  update latest render function spec (#372)
  pare down react jsx vs template comparison

# Conflicts:
#	themes/vue/_config.yml
Peter-WF referenced this pull request in Peter-WF/vuefe.github.io Nov 18, 2016
* add trim modifier

* fix grammer and typo

* re-fix grammer

* fix typo
Peter-WF referenced this pull request in Peter-WF/vuefe.github.io Nov 18, 2016
* 2.0:
  add trim modifier (volksbright#374)
  Update vue_version
  add instance methods to API docs
  add directives to API docs
  remove outdated and nonsensical example.
  Remove ElementDirective example, as this functionality was deprecated.
  slim down lists of referenced companion libs
  update one more function component slots syntax example
  update slots syntax for render function
  update latest render function spec (volksbright#372)
  pare down react jsx vs template comparison

# Conflicts:
#	themes/vue/_config.yml
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