Thomas Burette• 190
@tburette
Posted
Nice solution!
Your CSS could follow BEM more. You did <div class="card sedans"> but you could have done <div class="card card--sedans">. See Why the modifier CSS classes are not represented as a combined selector?
When the button becomes active the card moves a little. Is it intentional? It could be a neat effect. If it is not : you add a border on :hover/:focus. It causes the layout to change.
Marked as helpful
0
P
Kamran Kiani• 600
@kaamiik
Posted
Sorry for the late reply. Yes, you’re right about the class name, I changed it. Also, I added a fixed height to the button to fix when hover. Thanks for your feedback. @tburette
0
