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

Ashton's Rolodex #29

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Ashton's Rolodex #29

wants to merge 5 commits into from

Conversation

ashtn
Copy link

@ashtn ashtn commented Jun 5, 2017

ROLODEX

Congratulations! You're submitting your assignment!

Comprehension Questions

Question Answer
What role does the Model play in Backbone? The Model plays a similar role that the model played in rails, it's in charge of data
How do Backbone Views compare to Rails controllers? They are very similar in that they both handle events, where rails handles crud events and views handle user events
How do Backbone Events work in comparison to DOM events? They work very similarly, in that they both require a listener and an event handler, the difference being that you can customize Backbone event, while DOM events you cannot.
What do you think of Backbone in comparison to raw JavaScript & jQuery? Backbone provides a bit of structure that is missing with raw JavaScript & JQuery
Do you have any recommendations on how we could improve this project for the next cohort?

@CheezItMan
Copy link

ROLODEX

What We're Looking For

Feature Feedback
Core Requirements
Git hygiene Remember to commit!
Comprehension questions Check
Functionality
Created a Contact Model Check
Created a Rolodex Collection Check
Created a ContactView which renders an individual contact Check
ContactView responds to a click event when the user clicks on the contact Check
RolodexView created which renders the list of contacts Check
DOM Events handled for creating new Contacts Check
The RolodexView responds to custom Backbone event generated by ContactView to show the modal Check
Styling, Foundation grid layout Check
All dynamic content is rendered using an Underscore template Check
Overall Well done! Only one note, avoid using raw jQuery ($('...')) instead use this.$.

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