Ovidiu-Antonio

Latest comments

• 

  Aaron Gonzalez•445

  @aaronpaulgzSubmitted about 4 years ago

  

  Intro component with sign up form

  1

  10
  

  Ovidiu-Antonio•3,125

  @ovidiuantonioPosted about 4 years ago

  Hello Aaron,

  This solution looks pretty good! There are a few tips that can help you make the solution even better:

  • add the background image to the body element instead of the html
  • you can add max-width: 1440px; and margin: 0 auto; to the main element to keep the content in the center of the page no matter how big the screen is
  • you can make the form a little bit larger for the tablet version

  In rest, all good, the site is responsive and the error messages are present (you can add email validation to display another error message if the email isn't valid). Good job!

  Happy coding and keep going!

  0
• 

  Mesut Çifci•485

  @mesutcifciSubmitted about 4 years ago

  

  Ping Coming Soon Page

  2

  10
  

  Ovidiu-Antonio•3,125

  @ovidiuantonioPosted about 4 years ago

  Hello Mesut,

  You did a very good job with this project! The site is responsive and it looks very good. The buttons have hover states and the error message is displayed properly.

  One thing you can do is giving the max-width property to the wrapper instead of the body (on a bigger screen the wrapper remains on the left side, because of that)

  Happy coding and keep going!

  1
• 

  Thanh Nhi•155

  @lttn-16Submitted about 4 years ago

  

  HTML CSS Animation

  2

  30
  

  Ovidiu-Antonio•3,125

  @ovidiuantonioPosted about 4 years ago

  Hello Thanh Nhi,

  Nice solution! You did a very good job! There are a few tips to improve your solution:

  • add max-width: 1440px; and margin: 0 auto; to the container, to keep the content looking good on larger screens too and consider adding the background images to the body instead of the container
  • hide the horizontal scroll, so it not show up when the animations play

  Happy coding and keep going!

  2
• 

  Jaronimas•375

  @jaronimas-codesSubmitted about 4 years ago

  

  Challange with Sass, BEM, @media and grid

  2

  20
  

  Ovidiu-Antonio•3,125

  @ovidiuantonioPosted about 4 years ago

  Hello Jaronimas,

  Very good job! Your solution is looking very nice (i like the animations on the cards) and is responsive.

  To get rid of the horizontal scroll remove the width from the body element. You can make the gap between the columns smaller by defining the gap just for rows: grid-gap: 1.5rem 0; (1.5rem for rows, 0 for columns), and if you want an even smaller gap just lower the width of the container or increase the width of the cards.

  Happy coding and keep going!

  1
• 

  Guliye•290

  @guliye91Submitted about 4 years ago

  

  Huddle Landing Page using Flexbox

  1

  20
  

  Ovidiu-Antonio•3,125

  @ovidiuantonioPosted about 4 years ago

  Hello Guliye,

  You did a nice job on this challenge! I have a few tips for that'll help you improving your solution:

  • replace the current width of the .container with max-with: 90rem; to keep the content in the center of the page, you can also do that with the header but don't forget to add margin: 0 auto; or you can add all the elements inside one div that will have max-width: 90rem; and margin: 0 auto;
  • add hover states for the footer links too, for a better ux, and try playing with transitions for smoother effects
  • for the mobile version, don't forget to add align-items: center; to align all the elements

  Happy coding and keep going!

  1