Blog Card - @Media Multi Screen Only HTML, CSS

Community feedback

• 

  Pierce Issah•120

  @PJIceskullPosted 11 months ago

  Hey there, you did a pretty good job with your solution, but you're missing your hover selection for the <h1> and box-shadow.

  I also don't think you have added margins to your <main> since you already using Flexbox in your body.

  Another nitpick from me is that I think that the border should only be 1px instead of 2px.

  Marked as helpful

  0
• 

  Maria Valentinova Ivanova•220

  @MariaValentinovaPosted 11 months ago

  I think you could optimize your code. I don't understand why you're accommodating so many screen size options. In the documentation, it says it's intended for mobile screens of 375px and desktops of 1440px, I believe. This way, there would be much less code. You should pay more attention to proportions: border-radius, and margins. Good use of the CSS flex property!

  Marked as helpful

  0
• 

  michiganprogrammer•40

  @michiganprogrammerPosted 11 months ago

  one thing you missed was making the learning button change color when hovered over.

  You can use :hover selector in css to achieve that.

  Hope that helps

  0