Responsive landing page using SASS, JS, CSS and HTML
Design comparison
Solution retrospective
I look forward to your comments 💚
Community feedback
- @VCaramesPosted about 2 years ago
Hey @CesarGMEFA, great job on this project!
Some suggestions to improve you code:
-
For accessibility purposes, its better to use rem/em instead of px for your CSS property values.
-
To help keep your CSS code organized and easier to use, I suggest implementing CSS Variables. This will come in handy when building large websites, using light/dark mode, etc…
It’ll look something like this:
:root { --primary-color: value; --secondary-color: value; --tertiary-color: value; }
And to use the variables you’ll use the var() function. So it’ll look like this.
h1 { color: var(—primary-color); }
Heres are some articles regarding CSS Variables.
You can also take a look at my projects and see how I use it.
https://developer.mozilla.org/en-US/docs/Web/CSS/Using_CSS_custom_properties
https://www.w3schools.com/css/css3_variables.asp
Happy Coding!
Marked as helpful1 -
- @anikahammedkhanPosted about 2 years ago
Nice Work 👌❤️ Design is perfect 👌 Color combination is good 👌
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