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

Queues - rolodex - Tamiko Terada & Tehut Getahun #25

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

Conversation

TamikoT
Copy link

@TamikoT TamikoT commented Jun 5, 2017

ROLODEX

Congratulations! You're submitting your assignment!

Comprehension Questions

Question Answer
What role does the Model play in Backbone? Backbone Models function like most models in MVC frameworks and helps us store and manipulate data more easily. From the Backbone framework we can inherit a Model object which has built-in functionality around interacting with data, setting validations, adding default values, reading data from an API, and triggering events when data changes.
How do Backbone Views compare to Rails controllers? In Rails, the Views portion contains HTML files as layouts to what the user will see. In Backbone.js, the Views more closely resemble Rails Controllers because they house logic around how the user will interact with the models from the browser.
How do Backbone Events work in comparison to DOM events? Backbone Events are events that are triggered when Backbone Model objects change, while DOM events occur when there is an interaction or change to the document displayed by the browser. Whenever data that is stored as an instance of a Backbone Model object is created, updated, or deleted, a Backbone Event is triggered.
What do you think of Backbone in comparison to raw JavaScript & jQuery? Backbone is a framework. JavaScript is a language. jQuery is a library. Both Backbone and jQuery are written in JavaScript. Both Backbone and jQuery make life easier. To use the Backbone framework we need to also use jQuery because it has a dependency.
Do you have any recommendations on how we could improve this project for the next cohort? We believe this project lends itself well to being a project for pairs. It worked out really well that we were working on this collaboratively because we could gain a deeper understanding of how Backbone worked by mapping out the relationships out loud. Even though it was an individual project, it seemed like people were organically problem-solving in pairs or small groups.

@droberts-sea
Copy link

ROLODEX

What We're Looking For

Feature Feedback
Core Requirements
Git hygiene yes
Comprehension questions yes
Functionality
Created a Contact Model yes
Created a Rolodex Collection yes
Created a ContactView which renders an individual contact yes
ContactView responds to a click event when the user clicks on the contact yes
RolodexView created which renders the list of contacts yes
DOM Events handled for creating new Contacts yes
The RolodexView responds to custom Backbone event generated by ContactView to show the modal no - Currently, your ContactView.showDetails() method pops up the modal directly, even though the modal is under the RolodexView's element. While this certainly works, it makes your code more delicate and difficult to follow - imagine if instead of two types of views you had 10 or 15, all touching each others' elements. A cleaner way to approach it would be to trigger a custom Backbone event on the ContactView, and listen for that event from the RolodexView.
Styling, Foundation grid layout yes
All dynamic content is rendered using an Underscore template yes
Overall

Good work overall! Make sure to spend some time practicing with custom backbone events on the final VideoStore project (and feel free to come ask if they're still fuzzy).

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