@LucasJCFreireSubmitted 7 months ago
P
Joan
@jrosellrAll comments
- @jrosellrPosted 7 months ago
This solution gets pretty close to the original. However, there are two points that could be improved:
- The subtitle is slightly smaller and has a typo.
- The distance between each link is slightly smaller.
A minor gripe that I have is that the README is written in Portuguese. It would be better if it was written in English to reach a bigger audience.
Marked as helpful0 - @abuzarcodesSubmitted 7 months ago@jrosellrPosted 7 months ago
This solution doesn't replicate the original design well enough.
I found some problems:
- Layout: it is not responsive
- Font family: uses Arial instead of Figtree
- HTML: strange tag choices, uses h2 for the author's name
0 - @lsilva021Submitted 7 months ago@jrosellrPosted 7 months ago
This solution is pretty good. The only thing that is missing from the original design is the drop shadow of the card.
A problem that I found is that the code is written in Portuguese. Next time consider writing it in English so that more people can understand it.
Marked as helpful0