id attribute value must be unique
<img src="images/icon-sedans.svg" class="sedansicon" id="sedansicon" alt="illustration of sedan vehicle" width="75">
Learn more Looking to hire developers?
<img src="images/icon-sedans.svg" class="sedansicon" id="sedansicon" alt="illustration of sedan vehicle" width="75">
Learn more I found the buttons a little bit difficult while building the project, specifically their width. I used a percentage measurement that seemed to go well as far as responsiveness, but is there a best practice?
I found this to be a fairly straightforward exercise in Flexbox, which I seem to be getting the hang of. I also changed the original orange color to a darker hue because an accessibility check yielded a low contrast warning.
Account Deleted
Hello C Lewis, I hope you are doing well!
About the buttons, I think those elements would redirect the user to another page and act as a links rather than as a buttons triggering some JS behavior. Using links instead of buttons in this case is a best practise plus are easier to style than buttons. You can find more info about links-buttons here: css-tricks link
Ps: I like the font change! Good job and keep going!
Marked as helpful
1
@alexcumplido thank you!
Are you seeing the font as different than what the exercise intended? I used the fonts that were indicated in the exercise files, and it looks the same to me. Just curious!
0
Account Deleted
Hello again C. Lewis! It is the same font indeed. It seemed different first time I checked in, don't know why.
0
Join over 180,000 people taking the challenges, talking about their code, helping each other, and chatting about all things front-end!