kct9281• 270
@kct9281
Submitted
Reza1878
@Reza1878
All comments
- Reza1878• 80
@Reza1878
Posted
The result is good, but the repository url is wrong i think. i got 404 not found error when click it.
0 - Abolaji Nofiu• 40
@JOAN-96
Submitted
What are you most proud of, and what would you do differently next time?
I am proud the time it took me to finish the task as I finished it much faster than I expected. If I were to do it again, I will make good use of the CSS flexbox.
What challenges did you encounter, and how did you overcome them?
The challenge I had was aligning all the items to the center and I overcame them by:
- using the flex property,
- directing the flex to column, and
- aligning the items to the center.
Reza1878• 80@Reza1878
Posted
The result is nice, but here's some point you can improve in your code:
- The font is not loaded on my side.
- Adjust the card placement to the center of the screen, you can use flex box in this case.
- Add alt attribute on img tag, so if the img failed to load it'll show the alternative text
0 - Erdenebulgan• 20
@bulgaagithub
Submitted
Reza1878• 80@Reza1878
Posted
The solution is good, the result is really close to design. But it'll be better if the img tag have alt attribute, so if the image is failed to load it will show the alternative text
Marked as helpful
0 - Shreyas235• 80
@Shreyas235
Submitted
Reza1878• 80@Reza1878
Posted
The solution is good, the result is similar with the design. It looks good in both small and large screen.
Marked as helpful
0
