Design comparison
SolutionDesign
Solution retrospective
Questions!
- I'm familiar with the "Don't repeat yourself" mantra, however, I had difficulty making the button:hover effect applicable to the .sedan .SUV and .luxury divs. How were you able to do this?
- How would you simplify the scss to reduce repetition?
- How would you improve accessibility here?
Community feedback
- @catherineisonlinePosted almost 2 years ago
Nice! 🙌🏻
I would also add some transitions for active states (when colors change on hover). It creates more interactivity and makes the project looks cooler. Active states can be done on buttons, links, titles which act like links or anything else, you choose.
You can read more about it here, in case you haven’t done much of it: https://www.w3schools.com/css/css3_transitions.asp
IF THIS WAS HELPFUL PLEASE MARK IT AS HELPFUL 🤩
0 - @wundaabPosted almost 2 years ago
- Your code: h1 { font-family: "Big Shoulders Display", sans-serif; font-size: 40px; font-weight: 700; color: hsl(0deg, 0%, 95%); text-transform: uppercase; margin-bottom: 1.5rem; }
Font family should be like this instead: font-family: 'Big Shoulders Display', cursive;
- What difficulty did you have in making the button:hover effect applicable to the .sedan .SUV and .luxury divs?
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