Responsive landing page using FlexBox

Community feedback

• 

  Raymart Pamplona•16,100

  @pikapikamartPosted about 3 years ago

  Hey, awesome work on this one. Layout in desktop looks great.

  Well on this one, flexbox is more suited since for example, on the section of the pricing plan, it is only a one row layout which flexbox is good at, though you can use grid as well on that and it will be just one row ( for me ). I find it hard to tell pros and cons in here since the layout is not really a hard one though.

  Some other suggestions would be:

  • I wouldn't add a min-width: 375px on the body as there are lots of phones that the width is much smaller or the same as that. Removing that will be great so that the layout will really adapt.
  • Background-image of the wave is missing, you might want to check that one out.
  • Good choice to use heading tag on the annual-plan but the heading level of choice is incorrect. It should have been h2 and not h3. Because you don't want to skip a level. If you use h3 make sure that h1, h2 are present "before" it.
  • The text below the annual-plan is not a heading at all , since it is the content that the heading tag (annual-plan) is referring to in the section. A p tag would be nice .
  • You don't have to add a text-element inside the a tag, just put the text directly as the a tag's text-content. Remove h5 inside.

  Aside from those, great job again on this one.

  0