Carlos Damian Perez• 1,810
@caarlosdamian
Posted
Hello, excellent work I liked your solution, I would like to give you some feedback
- Enclose your main content in a
<main></main>
tag - Instead of having your application
src/assests/componentets
, I would suggest usingsrc/sections
, since they are sections and not components example of is a testimonial card. - Since you use typescript try to be more descriptive in the name of the properties and add the type when you are making a map.
- Instead of just using
<divs>
try using<section>
so it's more accessible - Use .ts instead of tsx when you are not rendering HTML
- Will be better to store yout data in src/utils/data.ts not tsx since your not rendering html
amazing work keep it up!!!
Marked as helpful
1
ronaldlamdev• 430
@ronaldlamdev
Posted
@caarlosdamian thank you !!! :)
0