KamronbekBotirjonov• 160
@KamronbekBotirjonov
Posted
Hi! Bro, your work came out a little rough. I think you need to reduce the button padding and margin from the bottom. I hope I helped you.
Marked as helpful
0
@KamronbekBotirjonov
Posted
Hi! Bro, your work came out a little rough. I think you need to reduce the button padding and margin from the bottom. I hope I helped you.
Marked as helpful
@mkboris
Posted
Nice work mirodiljondev, here are a few things to review
main
tag around the .onea
(anchor) tag instead of the button
tag. The a tag is used for navigation to other pages, while the button tag is designed for interactive actions like submitting forms or for events like toggling content. Also, using an unordered list ul
to group the social media links is a better approach for both semantics and accessibility.heights
and widths
on elements, as this can create problems with responsiveness and content fit. Instead, let the content and padding determine the element’s size. If necessary, use max-width
or min-height
, and prefer relative units like rem
for better adaptability. Change the width
of the .three to max-width
and it should be defined in rem
. Also remove the height
completely. head
section rather than directly in your CSS file as it enables asynchronous downloading, improving page load times.Hope this helps
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord