Responsive QR Code Component using grid

Community feedback

• 

  Manuel Gil•340

  @ManuGil22Posted over 2 years ago

  Hey @Danny-Rodriguez pretty cool solution! Have only one thing to say as feedback:

  • Always use heading tags in order. Dont use <h4> as a first tag, you can use a <h1> instead and change the text-style with css!

  Marked as helpful

  2

  ---

  

  Danny•10

  @Danny-RodriguezPosted over 2 years ago

  @ManuGil22 Thanks, I figured my way of doing that was hacky

  0
• 

  Prince Roy•1,210

  @iprinceroyyPosted over 2 years ago

  Please use the main element to wrap all the contents of a page & wrap class attribution elements inside the footer element. This way you can avoid html accessibility issue. Adjust padding/margin for mobile view as contents of card are collapsing. Happy coding :)

  0