
Please log in to post a comment
Log in with GitHubCommunity feedback
- @Kein-Internet
There's very little to say about this one, the HTML and CSS code are good for the most part, very nice.
A couple of suggestions would be to use seperate the flex code as it's own utilty class and just applying that class to the necessary components, so avoid you repeating the same chunk of CSS for each element. I would also use
rem
instead ofpx
when setting the margin/gap, padding and font-size so that they accomodate the users defult broswser font-size. Doing that is necessary for making an accessible site. - @ClemFrance
Nice one your Code is quite impressive and admiring meanwhile I think u could have added the button simply by using button tag which could have being more easier instead of making a 'link' div
- P@aysenurtatli
There is a problem in centering the design but good job.
- @LuRamirezR
Your code functions correctly, allowing you to reach the target. You should use variables for the colors, so if you ever need to change the page's color scheme, you only have to do it once. For example: :root { --bg-color-body: #141414; --bg-color-main: #1f1f1f; --bg-socialmedia: #333333; --location: #c5f82a; }
Join 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