Responsive flexbox card

Community feedback

• 

  Bhavya Bhandary•10

  @bhavyabhPosted almost 3 years ago

  Hi Heidel,

  Good job with the solution!

  Some of the suggestions:

  1. It is a good practice to put all the CSS related codes in a separate .css file and then link it through the html file.
  2. The page should contain one h1 heading. So, changing <h2>Order summary</h2> to <h1>Order summary</h1> will resolve the accessibility issue.
  3. The possible values for the font-style is 'normal/italic/oblique/initial/inherit', if you want to bold the text you can use font-weight: bold. It would resolve the HTML validations issue.
  4. Background issue can be resolved by: background: no-repeat top/100% url("../images/pattern-background-desktop.svg"); background-color: hsl(225, 100%, 94%);

  Keep coding!

  Regards, Bhavya

  0
• 

  Hania B.•1,320

  @techantherePosted almost 3 years ago

  Please try this solution: background: url("images/pattern-background-desktop.svg") no-repeat top hsl(); or use separate properties instead of shorthand background:

  background-repeat:no-repeat;
  background-position:top; 
  background-size: contain;
  

  0