Skip to content

Week 2 Homework Lynne Cooney#3

Open
ilynne wants to merge 22 commits intofrontend-application-development-uw20:masterfrom
ilynne:develop
Open

Week 2 Homework Lynne Cooney#3
ilynne wants to merge 22 commits intofrontend-application-development-uw20:masterfrom
ilynne:develop

Conversation

@ilynne
Copy link

@ilynne ilynne commented Apr 19, 2020

Week 2 HW Submission

Please fill out the information below in order to complete your assignment. Feel free to update this comment later if necessary.

  • Comfort rating on this assignment: 4
  • Completion rating on this assignment: complete

  • Adds FontAwesome React library for icons.
  • Uses grid and flexbox for layout.
  • Does not attempt to exactly match layout screenshot, but comes close.
  • Adds some responsiveness to layout.
  • Cards are clickable.
  • Bookmarks toggle when clicked using state.
  • Deployed to https://ilynne.github.io/week2-assignment/

Note that I changed the default port to 4000. I use 3000 for a lot of work projects and it's just easier for me to use 4000 for class projects.

"node-sass": "^4.13.1",
"react": "^16.13.1",
"react-dom": "^16.13.1",
"react-scripts": "3.4.1"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note for next time, don't forget to include prop-types here as well.

"@testing-library/jest-dom": "^4.2.4",
"@testing-library/react": "^9.3.2",
"@testing-library/user-event": "^7.1.2",
"node-sass": "^4.13.1",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

}

ArticleList.propTypes = {
className: PropTypes.string.isRequired,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great job making this reusable in both sections

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