@takinabradley
Posted
Great job, especially considering you didn't have a figma file! Much closer than I've normally gotten on these so far.
Everything looks great for the most part. I like how you made the component reusable by making it accept a prop, though perhaps it would be better if the parent component fed the actual data into it.
I'm not sure how I feel about the empty Header component. Perhaps that could have been part of the App HTML/styling?
Marked as helpful
@shalri
Posted
@takinabradley Thank you so much for the feedback. I am still feeling my way around React and I appreciate your suggestions.
Regarding the empty Header component, I just forgot to delete it from my template file.
Cheers!