Few lines in js, flex and grid

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted almost 4 years ago

  Hey, DG00! 👋

  Congratulations on completing your first Frontend Mentor challenge! 🎉 Good work on this one! 👍

  A few things I suggest are,

  • Adding cursor: pointer to the cards in the "Creations" section.
  • Making sure the hover effect for the cards in the "Creations" section remains visible when the titles of the cards are hovered over.
  • Adding some more space between the navigation links in the logo of the page in the mobile menu.

  You can make the background of the hero section darker by adding a linear-gradient to the background of that section and setting both of its colors to black (with some opacity in it, so that the background image remains visible).

  Hope that helps. 😀

  Keep coding (and happy coding, too)! 😁

  1
• 

  Carlos T•160

  @ctx14Posted almost 4 years ago

  Nice job!

  1
• 

  Roc Tanweer•2,500

  @RocTanweerPosted almost 4 years ago

  Never settle! Your font size are way different than the design and header or hero image should be a bit darker

  hope it helps

  1

  ---

  

  DG00•20

  @dgebara00Posted almost 4 years ago

  @RocTanweer Thanks a lot for comment. You motivated me to improve that and I think it's way better now but still dunno how to make background darker.

  0
• 

  DG00•20

  @dgebara00Posted almost 4 years ago

  @ctx14 Thanks :>

  0