learn-co-curriculum / react-hooks-state-and-events-lab Goto Github PK

Canvas Link

https://learning.flatironschool.com/courses/5256/assignments/219969?module_item_id=512745

Concern

I think this lab should say that it will not pass unless you default start the app in "light mode" vs "dark mode". Thanks!

Additional Context

No response

Suggested Changes

No response

Thanks for raising this issue! Future learners thank you for your diligence. In
order to help the curriculum team address the problem, please use this template
to submit your feedback. We'll work on addressing the issue as soon as we can.

Please fill out as much of the information below as you can (it's ok if you
don't fill out every section). The more context we have, the easier it will be
to fix your issue!

Note: you should only raise issues related to the contents of this lesson.
If you have questions about your code or need help troubleshooting, reach out to
an instructor/your peers.

Link to Canvas

Add a link to the assignment in Canvas here.

What should be changed?

Nowhere in the instructions does it mention that you also need to update the class on the button in the Item.js file, so that you can have the button go between "Add to Cart" and "Remove from Cart" in order to make the project look like the GIF included on Canvas.

Suggest your change here. Let us know what section or line of the Readme needs
to be updated, and any proposed corrections. Include the line number(s) if
possible.

Additional context

Add any other context about the problem here.

Link to Canvas

https://learning.flatironschool.com/courses/2674/assignments/123703?module_item_id=255315

What should be changed?

The readme references a demo students should follow, but doesn't indicate what or where the demo is.

This lesson: https://learning.flatironschool.com/courses/4600/modules/items/310615 doesn't have a button on it to report issues, so I'm reporting it here. In the first lesson in the playlist, the instructor replaces one image URL with another, but the second URL doesn't work anymore, so he is explaining all the features of an image that we can't see. The issue appears at around 1:40.

Canvas Link

https://learning.flatironschool.com/courses/4609/assignments/150073?module_item_id=318041

Concern

For the "Add to Cart" section, it instructs that the <li> className switch between "in-cart" and "". However, in "./src/data/index.css" I found that this wasn't correct, as it showed:
`.Items li.in-cart {
background: var(--background--light);
}

.Items button.add {
background-color: var(--yellow);
}`

Additional Context

Suggested Changes

All that is needed to fix this issue would be to edit the content of the code snippet.

Canvas Link

https://learning.flatironschool.com/courses/6558/assignments/242873?module_item_id=571386

Concern

Although my program so far matches the DEMO, the DEMO has a bug. Which means my code has a bug right now. Pay Close attention to Pomegranate and Lettuce.

*Yogurt
*Pomegranate
*Lettuce
String Cheese
Cookies

*in cart initially

NOW filter out all but dairy:

*Yogurt
String Cheese

NOW show all:

*Yogurt
Pomegranate
Lettuce
String Cheese
Cookies

ALTERNATIVELY:

*Yogurt
*Pomegranate
*Lettuce
String Cheese
Cookies

*in cart initially

NOW filter out all but dessert:

Cookies

NOW show all:

Yogurt
Pomegranate
Lettuce
String Cheese
Cookies

AS THE USER, I DID NOT REMOVE THE ITEMS FROM THE CART.

Additional Context

GOAL IS TO MATCH DEMO. DEMO HAS A BUG WHERE WHEN YOU SHOW ONLY ITEMS IN A CATEGORY THAT DOES NOT HAVE THE ITEMS IN THE CART, THE ITEMS IN THE CART ARE REMOVED FROM THE CART WHEN THE FILTER REGENERATES THE ITEMS.

Suggested Changes

Instead of filtering out the items, we could say style.display="none" for the items to be excluded; and of course style.display = "block", but in React it would be different because you must change the display state to get the item to regenerate...