Any feedback?
Basile Raiwet
@BasileRaiwetAll comments
- @Zana994Submitted almost 4 years ago@BasileRaiwetPosted 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 -
- @SamyrORSubmitted almost 4 years ago
Hello everybody, i expect you have a nice day.
Here go another challenge, all feedback well be welcome.
@BasileRaiwetPosted almost 4 years agoHi 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 - @domiattawaySubmitted 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
@BasileRaiwetPosted almost 4 years agoHello 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 - @bobaramSubmitted 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 .
@BasileRaiwetPosted almost 4 years agoYou 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