All comments

• 

  Krishna Talekar• 30

  @krishnatalekar

  Submitted 6 months ago

  

  Product preview card component responsive design using CSS Grid & Flex

  1

  20

  

  Ivandro Neto• 160

  @ivandro-neto

  Posted 6 months ago

  Hey! I appreciate your solution, but I suggest instead of using rem units for your main-container height property use "dvh" units to make your layout most responsive for any screen.

  Have a happy code!👋🏿

  Marked as helpful

  1
• 

  Zenos• 190

  @Zenosaki

  Submitted 7 months ago

  i faced many difficulties i m sad a little but no probleme: 1)i didnt know what to do for the tick list and their content i tried everything but the flex bow and the aligne didnt work

  1. again i tried to mesure the size of the fonts but no solution i keep getting a different font size from the original

  2. in the success box i didnt know how to put a gap or a spacing between the lignes in the paragraph

  3. this is the main difficulty : i tried the media query but i didnt know what to do i keep getting a random designe i dont know why and how to change the photo from the desktop one to the mobile one and again it didnt work i tried undesplaying one and keeping the other but no unluckly

  =>just some last word i loved this challenge even if i didnt made it as it should be i did this challnge for 2 days i m happy with it but i need help i really need help and how to use media queries correctly

  

  Newsletter sign-up form with success message

  1

  60

  

  Ivandro Neto• 160

  @ivandro-neto

  Posted 7 months ago

  Hi! I just noticed that you hadn't finished the responsive part of your solution, then there some tips to improve it:

  • Adjust the font size of each text tag;
  • Change the ".container" width to 100% on mobile layout, and change the flex-direction to "column";

  Have a happy coding!❤️

  Marked as helpful

  0

  This comment has received 1 replyShow
• 

  Michael R. Thompson• 10

  @14thommi

  Submitted over 1 year ago

  

  QR Code Component w/ Flexbox

  1

  30

  

  Ivandro Neto• 160

  @ivandro-neto

  Posted over 1 year ago

  Hi there, 👋 Your solution looks great, but instead use <section>...</section> as a main container, HTML has a specific tag for this called <main>...</main>. You must change this one for accessibility best practice.

  Have a happy code.❤️

  1
• 

  Ivandro Neto• 160

  @ivandro-neto

  Submitted over 2 years ago

  

  Responsive lading page using Media Query and FlexBox

  2

  70

  

  Ivandro Neto• 160

  @ivandro-neto

  Posted over 2 years ago

  Hi, I would like to recieve your opinion and tips to improve my code.

  0