All comments

• 

  Melissa-Sadeg•70

  @Melissa-SadegSubmitted 10 months ago

  It's my first try on frontend development, all comments are welcomed !

  

  QR-Code-Composant

  5

  40
  

  Bowen•280

  @bowen-weiPosted 10 months ago

  Your solution looks great!

  0
• 

  Karvendhan•270

  @imkarvendhanSubmitted over 1 year ago

  Hi, Give Some Feedback To Improve my Code at Next Time

  

  Huddle Landing Page With Single Introductory Section

  1

  30
  

  Bowen•280

  @bowen-weiPosted over 1 year ago

  hey 👋, great job, but looks like you didn't center the elements inside your main section you can do this with :

  /* Main Section */
  
  
  main{
  min-height: 100vh;
  display: flex;
  flex-direction: column;
  justify-content: center;
  
  background-color: hsl(257, 40%, 49%);
  background-image: url(../images/bg-desktop.svg);
  background-repeat: no-repeat;
  }
  

  Hope that helps you

  0

  This comment has received 1 replyShow
• 

  MHAA•50

  @MHA2001Submitted over 1 year ago

  Hi guys! I ran into a problem with the project when I tested the result on my mobile. There was a lot of white space at the top of the page, while I don't see any white space in Chrome's developer tools. Do you know a better tool for simulating the site on mobile and how can I eliminate this empty space?

  

  results summary with flexbox and mediaquery

  1

  40
  

  Bowen•280

  @bowen-weiPosted over 1 year ago

  Hello, your solution looks great !

  I think maybe you can change "align-item" of the body to "flex-start", and in the desktop setting(your media query), change it back to "center".

  Adding box shadow may make it look better .

  Marked as helpful

  0