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 - Anna Barklund - Rolodex #30

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

Queues - Anna Barklund - Rolodex #30

wants to merge 15 commits into from

Conversation

amb54
Copy link

@amb54 amb54 commented Jun 5, 2017

ROLODEX

Congratulations! You're submitting your assignment!

Comprehension Questions

Question Answer
What role does the Model play in Backbone? Taking care of all things related to the data. Create, update, delete as well as including functions that are related to data. Triggers events when data changes.
How do Backbone Views compare to Rails controllers? About the same. It is the link between the DOM and the model. When an event is happening in the DOM, the view communicate this to the model (if needed) and if something is changed in the data, the view communicates this to the DOM by updating it.
How do Backbone Events work in comparison to DOM events? The backbone events is triggered in the model and is listened to in the view.
What do you think of Backbone in comparison to raw JavaScript & jQuery? I like the views that organize the different functions to use for the view. It is easier to understand the flow. I like the way to organize all events in the events hash.
Do you have any recommendations on how we could improve this project for the next cohort? I really like the live code, but if it is possible to git add/commit more often it would be great. Then one can find out afterwards how the code was refactored. An option would be that we the students ask for it, but sometimes, at least I am still in the last phase finishing up, when the things happening on the screen is well into the next step, and the moment to ask is gone.

amb54 added 15 commits May 30, 2017 18:41
…el Contact. Created a collection of contacts with only one contact (contactData). Created an instance variable oneContact of the model Contact. Created the .ready block with two compiled templates that also is used.
…he model as well as assigning backbone model Contact.
…e variables and one vareable for a collection. Created the methods (functions) readForm, renderContact and renderRolodex. Updated the .ready block to handle adding a new contact and rendering all the contact cards.
…RolodexView as anobject with the properties initializer, render, events, addContact, clearForm and readForm.
…mented out all codewith functionality now incorporated int the views RolodexView and ContactView.
… the trigger function for rolodex_view to listen to.
@PilgrimMemoirs
Copy link

ROLODEX

What We're Looking For

Feature Feedback
Core Requirements
Git hygiene Well Done
Comprehension questions Well Done - thanks for the feedback!
Functionality
Created a Contact Model Well done - nice defaults
Created a Rolodex Collection Well Done
Created a ContactView which renders an individual contact Well Done
ContactView responds to a click event when the user clicks on the contact Well Done
RolodexView created which renders the list of contacts Well Done
DOM Events handled for creating new Contacts Well Done
Styling, Foundation grid layout Well Done
All dynamic content is rendered using an Underscore template Well Done
Overall
Great work. This submission meets all expectations and learning goals for this project.

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