Simon Capillonβ’ 100
@simon7195
Posted
Hi! π
Your solution is really nice, I really like the little animation that moves when the buttons are hovered.
May I make a few comments :
- For the avatar, you used the background of your
div
to insert your image. I think using animg
tag would have been more appropriate, especially in terms of accessibility, as in your case you can't display alternative text (alt="avatar of ..."
). - For geographic datas, a
p
tag would have been more appropriate than anh2
. - The
section
tags defines important areas of a site. In the case of a linktree, adiv
orspan
would have been more appropriate. - Regarding fidelity to the proposed design, your dimensions differ a little from the model. It's a detail, but for some designers and customers it can be annoying.
- Your CSS is very short, I think that's great, especially since on my side they're usually longer π. However, I think you should use classes because in a larger project, using tags as selectors is a dangerous gamble and makes it difficult to read your code (especially if it's uncommented as is your case).
I hope you find my comment useful. Keep it up! βοΈ
0