QR-code-component

Community feedback

• 

  Steven Portas•440

  @satropPosted about 2 years ago

  Nice work! Super clean HTML and nice CSS.

  Only have two suggestions. In the HTML layer, I would add a title in the head so your tab reads something a little more interesting than "Document". The other... more a thing to think about than a suggestion. From a front-end professional point adding images as a background-image in the CSS layer can be problematic for some CMS systems to handle. It would be better to have the image either inline on a div or as a standard img-tag either option is easy to style as though it were a background image with CSS/SCSS.

  All the best, Steve.

  Marked as helpful

  0

  ---

  

  adr99•410

  @leorichy99Posted about 2 years ago

  @satrop Thanks 😍

  1