All comments

• 

  Zana• 180

  @Zana994

  Submitted almost 4 years ago

  Any feedback?

  

  Responsive landing page using sass, flexbox, html

  1

  10

  

  Basile Raiwet• 175

  @BasileRaiwet

  Posted almost 4 years ago

  Hi Zana ! Great work ! here are some ideas to improve your solution :

  • the gallery section is a bit tight on my 13 inch device (boxes are touching each other). Maybe you can put some margin between each boxes (or use gap but gap is not well supported actually). If you want to try, you can use css grid to build this section. I think it will be easier.

  • you define your section width with width: 80% on your .frame class. The problem is that your container (and potentially his children) will extend too much on large screens. In most cases, it's better to set the width with a strict value (like on this website). In this way, the content will be centered and your website will be readable on any screen size

  1

  This comment has received 2 repliesShow
• 

  Samyr Oliveira Ribeiro• 215

  @SamyrOR

  Submitted almost 4 years ago

  Hello everybody, i expect you have a nice day.

  Here go another challenge, all feedback well be welcome.

  

  Responsive, GRID, SASS

  2

  11

  

  Basile Raiwet• 175

  @BasileRaiwet

  Posted almost 4 years ago

  Hi Samyr !

  Great work ! I just see some points you can improve :

  • About semantics : avoid to use many h1 on the same page. h1 and h2 are here for the title and subtitle of the section. So It's better if you use h3 (or other, depend of the situtation) for the boxes titles
  • about the container width : design is based on a 1110px grid and I think it's better to limit the width of the container to this size because if your solution stretch to 80% of the viewport with (like it is actually), the result could be really different on larger screens. (As an example, look how this page container is defined. It's, I think, the most common way to define container width)

  1

  This comment has received 1 replyShow
• 

  Dominique• 40

  @domiattaway

  Submitted almost 4 years ago

  I don't understand. When I do the live view of my work from Vs code it is working but when I upload it to vercel it is distorted. Looking ofr feedback. Thanks

  

  Four Card feature HTML-CSS

  2

  10

  

  Basile Raiwet• 175

  @BasileRaiwet

  Posted almost 4 years ago

  Hello Dominique !

  As Samyr said, check if your work is up to date on github and then re-deploy to vercel.

  I also have some recommendations for you :

  • Avoid relative position for your boxes. This solution is not really flexible. Instead, you can use flexbox or grid (grid is super useful and make positioning super easy for this exercice)
  • You can set the image position without using a margin or padding. (check how position relative / absolute work together in the child / parent relation.. It's super useful)

  0

  This comment has received 1 replyShow
• 

  Omen• 50

  @bobaram

  Submitted almost 4 years ago

  From the very start i struggled with this project . I need feedback on how i should have structured my HTML and which tags i should have used. I am still learning and would really appreciate any sort of feedback no matter how critical it is .

  

  single-price-grid

  3

  10

  

  Basile Raiwet• 175

  @BasileRaiwet

  Posted almost 4 years ago

  You can easily improve this solution by adding some padding / margin in order to fit with the design. Also, look at the design file to find correct font-size for your headings / content. don't forget details like border-radius and box-shadow.

  2