@walmanjm
Posted
hi,
if you wanna stick around with your " margin" way ,
you have to calculate the margin-bottom:
margin-top:
,
something like this :
in your code, you put review-1
review-2
review-3
inside reviews
,
the margin-bottom:
of review-1
is should be margin-top:
of review-3
,
and the margin-bottom:
margin-top:
of review-2
should be between ,
in code like this :
.review-1 {
margin-bottom: 2rem;
}
.review-2 {
margin-top: 1rem;
margin-bottom: 1rem;
}
.review-3 {
margin-top: 2rem;
}
we can say this the traditional way to what @alwodzicki suggestion with flexbox
by the way take a look also in mobile its need some correction,
and don't forget about the :
bg-pattern-bottom-desktop.svg
bg-pattern-bottom-mobile.svg
bg-pattern-top-desktop.svg
bg-pattern-top-mobile.svg
happy coding 👍👌
Marked as helpful
@AtanasovCode
Posted
@walmanjm Hi Walmanjm!
Thank you for taking the time to help me improve my method of using the margin-top
property to style and align the boxes. I ended up implementing @alwodzicki 's solution because as you mentioned too it is a bit better and makes the code easier to understand.
I also took note of the mobile design and I believe I have fixed the issues there so now it looks and functions a lot better. I just had to change some things and I got it to work properly :)
As for the background photos I didn't include them as I don't really seem to notice a difference with them and so I made the choice to leave them out, hope this isn't a very bad thing to do :)
Thank you again I really appreciate it :)