Intro component with grid, flex and js

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted almost 4 years ago

  Hey, Stelin! 👋

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

  I suggest,

  • Adding a max-width to the main container or wrapper to prevent the content from becoming too stretched on extra-large screens.
  • Adding some actual margin or padding around the top and bottom of the page so that there's always some room between the content of the page and the edges of the screen in the desktop layout.
  • Adding text-align: center to the text content of the page in the mobile layout.
  • Adding some more space between the form and the edges of the screen in the mobile layout.

  Keep coding (and happy coding, too)! 😁

  And happy holidays! 🎄

  1
• 

  Harold•628

  @harika09Posted almost 4 years ago

  Great Job on this one @Stelin

  Got some few recommendation.

  1. check the validation when I clicked the button it will just stack the error.
  2. add cursor: pointer; when hover on the button.
  3. lower the padding on the smaller device on class="info".

  Good job on this. Happy coding!

  1

  ---

  

  Stelin•10

  @StelinUwUPosted almost 4 years ago

  @harika09 Thank you for the feedback ❤

  0
• 

  Web Frog•1,840

  @MasterDev333Posted almost 4 years ago

  Great work @Stelin. Everything is fully rsponsive and close to design. One suggstion, title should be h1 tag, not span. :) Happy coding~

  1

  ---

  

  Stelin•10

  @StelinUwUPosted almost 4 years ago

  @MasterDev333 ❤

  0