@fidellim
Posted
Hi @LoganWillaumez,
Great work finishing the project! It looks great on both desktop and mobile views. Some suggestions I would like to share are of the following:
- You could try implementing semantic tags on your project. So that it gives meaning to it. It will also be easier for other developers or users to have a broad idea of what the content would be.
- You could add
cursor: pointer
on social media icons as it would appear that these can be clicked. - On mobile view, the social media icons are not shown when the right arrow key is pressed. You might want to have a look at that.
Other than those, great job :)
Marked as helpful
@LoganWillaumez
Posted
Hello @fidellim and thanks for your feedbacks and your help !
I corrected my project and I'll check some documentation for properly improve the meaning of my HTML :).
Have a nice day !
@fidellim
Posted
@LoganWillaumez That is great! You can have a look at this link regarding semantic tags. Have a nice day as well :)