Skip to content

Updated the demo to use v-class #62

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

Closed
wants to merge 1 commit into from
Closed

Updated the demo to use v-class #62

wants to merge 1 commit into from

Conversation

crswll
Copy link
Contributor

@crswll crswll commented Mar 11, 2015

I noticed you were using class="{{done ? 'done' : ''}}" instead of v-class="done: done". I wasn't sure if there was a particular reason for this but I figured I'd submit a pull request either way.

@yyx990803
Copy link
Member

The two are largely equivalent, but in this particular case I think the interpolation is less confusing. v-class is more useful when there are multiple classes to be toggled.

@yyx990803 yyx990803 closed this Mar 12, 2015
@crswll
Copy link
Contributor Author

crswll commented Mar 12, 2015

Sweet.

dingyiming added a commit that referenced this pull request Dec 21, 2016
lex111 pushed a commit to lex111/vuejs.org that referenced this pull request Feb 3, 2018
Управление повторным использованием элементов
isatrio pushed a commit to isatrio/vuejs.org that referenced this pull request Mar 9, 2019
* Translate Bahasa Indonesia : Permintaan Vue

* fixed typo and few things

* Update typo

* updated another typo

* updated translation of lifecycle hook

* Translate Bahasa Indonesia : Properti Penghitung & Pengawas

* fixed typos and translation

* Translate Bahasa Indonesia : Component Registrasi

* fixed translation

* fixed translation
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.

2 participants