Axda• 70
@Axda-Web
Submitted
Feedback appreciated 🙏
Zainab
@ZainabProgrammer
All comments
- Zainab• 180
@ZainabProgrammer
Posted
Greetings,
I appreciate your work and that was amazing to see how interactive this project is. Well done!
1 - An• 150
@anaislcno
Submitted
It was my very first utilisation of an API with React so I'm glad I did that. If you have any ideas for improving the code, especially for integrating the API and the little button, let me know! If you have any other ideas, I'd love to hear them too :)
Zainab• 180@ZainabProgrammer
Posted
Hey there,
Your design looks good on small screens but try to make card centered on larger screens and also try to add a loading state when api is fetching data to make it more interactive and engaging.
Hope it helps...
1 - Moises Castillo• 220
@mcasti19
Submitted
Hello, here I leave my 2nd challenge, I am starting, I have been learning little by little from different sources, but a community is always much more effective and efficient to improve. Any suggestion you can give me would be very welcome. Thanks"
Zainab• 180@ZainabProgrammer
Posted
Hi, there overall your code looks good and design too, there is suggestion from me that inside your paragraph you should align text centred instead of taking the full width and give the padding to make it as close to design as possible.
.track-item{ text-align:center; padding:1rem; }Hope, it help you..
0 - AdamsRuth1• 20
@AdamsRuth1
Submitted
I'm not so sure of my code responsiveness and i am really finding it difficult setting my code to adjust to various screen sizes
Zainab• 180@ZainabProgrammer
Posted
To make the div centred both horizontally and vertically you should try:
.main-div{ width:100%; height:100vh; display:flex; align-items:center; justify-content:center; }Hope it helps you...
Marked as helpful
0
