Wonski312• 100
@Wonski312
Posted
I think would you did is all good with sections etc but could be done without them at all because this project looks a bit like component so just divs with classes set accordingly would be enough.
As you can see in your report Section class payment button require h2 tag but I would say it’s not necessary in this section so with div you will avoid it
I think displayflex should be assigned to body element because you are positioning whats in body as parent element not the whole html file. After that add height:100vh and you probably will not need margin top on your body element which probably should be on content wrapper class.
Well done for your work and good luck with next one:)
Marked as helpful
0
