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

Karinna Iniguez - React Timeline - Octos #41

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

Conversation

karinnainiguez
Copy link

@karinnainiguez karinnainiguez commented May 29, 2018

React Timeline

Congratulations! You're submitting your assignment!

Comprehension Questions

Question Answer
Explain what purpose a component serves in React? A component allows us to apply a look and feel, as well as behavior to a part of the page. It also allows us to organize our page better and dry up our code by reusing and nesting components.
How does adding dynamic content to a web page in React compare to jQuery? It's similar because we still use functions and classes to apply behavior and style. But it's a lot more organized to apply that to a component.
How did you use props in this project? My TimelineEvent component had props that I needed to display, such as the person, status, and timestamp. By making those props, I was able to uniformly create those components and display them.
Do you have any recommendations on how we could improve this project for the next cohort? Maybe going over the process of using the map function on a collection to return components a little bit more than we did would be helpful.

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.

1 participant