responsive newsletter form using flex-box

Community feedback

• 

  Ranit Manik•880

  @RanitManikPosted 5 months ago

  Your solution looks legitimate. I have reviewed your code, and here are some suggestions from me:

  1. Specify Image Dimensions:

     • Always set both the height and width attributes for images. This practice ensures that the space required for the image is reserved when the page loads, preventing layout shifts. Your success page is currently experiencing layout shifts because these attributes are not specified. For more details, refer to this article.

  2. Display User Email:

     • On the success page, ensure you display the user's previously input email. Currently, the email shown is always [email protected], which is incorrect.

  3. Alt Text for Images:

     • Avoid using alt text for non-decorative images. In this challenge, all images are non-decorative, so you should leave the alt attribute blank, like this: <img src="./assets/image.jpg" alt="">. For more information on this best practice, watch this video.

  • I hope you find my feedback helpful. Please mark it as helpful if you do.

  Marked as helpful

  0

  ---

  

  Dipesh•430

  @Dipesh-sapkota1Posted 5 months ago

  @RanitManik Thank you for your feedback.I have another question. I've been using vanilla CSS for a while and have a basic understanding of responsive design. I know there's more to learn, but based on my project, should I focus on improving my skills with vanilla CSS, or should I start using frameworks?

  0

  

  Ranit Manik•880

  @RanitManikPosted 5 months ago

  @Dipesh-sapkota1 I think you should start using CSS frameworks as they are like CSS on steroids. However, under the hood, they are essentially just CSS. I suggest you begin using SCSS/SASS and then transition to Tailwind CSS.

  From Your code I think you are more than ready.

  0