HTML, SCSS, FLEXBOX

Community feedback

• 

  Hania B.•1,320

  @techantherePosted almost 3 years ago

  The solution looks great and responsive as well. I am adding few tips here:

  1. Use an anchor tag where you see a hover state, like in your solution instead of button use an anchor tag and style it the same way to look like a button, makes it semantically sound.
  2. Nested CSS is not good for specifying a class instead you can simply mention class name alone that will make code more efficient with less specificity, for example use .card-context alone instead of writing it with parent class, it is sufficient. I can't see hover states behavior as specified by design files, you should add them too. Everything else is fine. Hope it's helpful. Good luck :)

  0