Design comparison
Community feedback
- @Kein-InternetPosted 3 months ago
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.1 - @ClemFrancePosted 3 months ago
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
0 - @aysenurtatliPosted 3 months ago
There is a problem in centering the design but good job.
0 - @LuRamirezRPosted 3 months ago
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; }
0@MatPawlukPosted 3 months agoHey! 😊 I took a big break from studying and now I want to refresh my knowledge and see where I'm at. Currently, I'm using Sass, and I keep my color variables in a separate file. 👍 Cheers! 🙌
0
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