@chilldeleuze
Posted
Good job, you nailed the design.
There's a small issue with the buttons though: They would only work if the user clicks directly on the text. A click anywhere else on the buttons wouldn't open the link. You might want to get rid of the <div class="box-social">
and instead rely only on the <a>
.
It seems like you used BEM, or some variant. Maybe try to use double dashes between block and element, so the code is a little more readable (e.g. .box--profile
. I also think leaving an empty space between the css-selector and the opening curly brace is a good idea, since it increases readability.
Marked as helpful