HTML, CSS, CSS Flexbox

Community feedback

• 

  Ahmed Faisal•5,095

  @afrusselPosted over 3 years ago

  Make the box area bit wider. Other wise everything is good

  Marked as helpful

  3
• 

  T

  Chamu•13,130

  @ChamuMutezvaPosted over 3 years ago

  some issues have been raised in the report that needs fixing, they may include the following:

  • the main element is expected to be present in a site. Consider changing the div with a class of bg into a main element.
  • also considered important is a h1 heading element. Headings should ascend in order h1, h2, h3 etc without skipping headings.
  • an inline element like a span cannot be used as a parent of a block element , see example below. Either use a div instead of the span

  <span class="an-plan">
          <p><h4>Annual Plan</h4></p> // this is not allowed
          <p>$59.99/year</p> 
        </span>
  

  Marked as helpful

  1

  ---

  

  Daniel Andrew•110

  @DxnielXndrewPosted over 3 years ago

  @ChamuMutezva Thank you. You've taught me some new things. I appreciate the input

  0
• 

  Adrian Surigao•30

  @surigaoadrianPosted over 3 years ago

  The cta button doesnt have box-shadows and its bg color is purple rather than blue. You should also put line-height and padding to your texts. Otherwise everything is fine

  Marked as helpful

  1