Feedback appreciated 🙏
Zainab
@ZainabProgrammerAll comments
- @Axda-WebSubmitted about 1 year ago@ZainabProgrammerPosted about 1 year ago
Greetings,
I appreciate your work and that was amazing to see how interactive this project is. Well done!
1 - @anaislcnoSubmitted about 1 year ago
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 :)
@ZainabProgrammerPosted about 1 year agoHey 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 - @mcasti19Submitted about 1 year ago
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"
@ZainabProgrammerPosted about 1 year agoHi, 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 - @AdamsRuth1Submitted about 1 year ago
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
@ZainabProgrammerPosted about 1 year agoTo 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 helpful0