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

Maja Octos #33

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

Maja Octos #33

wants to merge 3 commits into from

Conversation

mgraonic
Copy link

@mgraonic mgraonic commented May 29, 2018

React Timeline

Congratulations! You're submitting your assignment!

Comprehension Questions

Question Answer
Explain what purpose a component serves in React? Components are like JS functions. They take inputs and return elements describing what should appear on the screen.
How does adding dynamic content to a web page in React compare to jQuery? React adds another layer of abstraction but handles better than jQuery (which can get messy quickly).
How did you use props in this project? Props are essentially the parameters for the components. I passed in parsed event data (and sometimes other components) in as props.
Do you have any recommendations on how we could improve this project for the next cohort? The waves were confusing. Clarify that we are doing a top-down approach (start by making the App.js file work) vs a bottom-up (start with more basic components). Change the naming convention of the files. React is confusing to begin with, and having 3 similarly named files (Timeline, TimelineEvent, and Timestamp) makes it hard to understand what you're doing.

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