-
Notifications
You must be signed in to change notification settings - Fork 4.7k
Update computed properties example #289
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@NataliaTepluhina This is great! Love the result.
Just had a few suggestions for consistency amongst the book titles, but otherwise you're good to go!
Co-authored-by: Ben Hong <[email protected]>
Co-authored-by: Ben Hong <[email protected]>
Co-authored-by: Ben Hong <[email protected]>
Co-authored-by: Ben Hong <[email protected]>
Co-authored-by: Ben Hong <[email protected]>
Co-authored-by: Ben Hong <[email protected]>
Would it be reasonable to add an Add book functionality to highlight that computed properties are in fact reactive? That could potentially help to understand why book list was put in |
Changed an example of
computed
properties to be less cryptic and more 'real-life'Close #38