Social links profile with a light or dark theme using Flexbox.
Design comparison
Solution retrospective
This was a basic challenge, so nothing was too difficult, but I decided to add a light mode as well along with some basic CSS transitions :).
What specific areas of your project would you like help with?There isn't anything specific that I would like help with, but general help or feedback would be much appreciated!
Community feedback
- @DarkstarXDDPosted 5 months ago
- Buttons are used for actions like toggling things, submitting forms etc.
<a>
should be used for navigation. The clickable elements on this design navigates the user to other websites. So they should be<a>
elements. Not buttons. You can style them to look like buttons. - Those set of clickable elements are also a list of items. So you should use
<ul>
and<li>
elements. - All the content should be wrapped in a
<main>
landmark element. Except the attribution, which should go inside a<footer>
. - font-sizes should be in
rem
. Not inpx
.
Marked as helpful2@yefreescodingPosted 5 months ago@DarkstarXDD
That’s exactly the advice I wanted to give him. You’re spot on with everything 😁👍
2@AffanRMPosted 5 months ago@DarkstarXDD Thanks a lot for the feedback! May I ask whether the above-mentioned points are good coding practices only or do they help with the SEO of the website as well?
0@DarkstarXDDPosted 5 months ago@AffanRM As for the
<a>
and<button>
, you can't have ahref
attribute inside the<button>
. So if you need to direct the user to another webpage or even a part of your own webpage you will have to use an<a>
.Rest of the points improve the accessibility of the website. Unlike a
<div>
or a<span>
, elements like<main>
,<ul>
,<li>
have meaning to them. They are semantic elements. When you use<ul>
and<li>
screen readers will announce that this is a list and those are list items. If you used 5<div>
elements instead of semantic list elements, a screen reader will not know that those are a set of list items. It will just read out each row as if they were new lines of a paragraph.Here is a good article that explains why you should use
rem
instead ofpx
.As for your question whether these will help with SEO? I think so. I have not dig deep into SEO, so I can't give a proper explanation but I am pretty sure web crawlers do benefit from the meaning the semantic elements give to the HTML.
Bottom line is, you should always try to use the most meaningful element for the task.
1 - Buttons are used for actions like toggling things, submitting forms etc.
Please log in to post a comment
Log in with GitHubJoin our Discord community
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