QR code component

Community feedback

• 

  T

  Grace•31,330

  @grace-snowPosted over 2 years ago

  Good job, this looks great!

  Small thing - when you make links open in a new tab, male sure you add rel="noopener" to it as well. It's necessary for security (you can look it up if you want to know more)

  Marked as helpful

  1

  ---

  

  P

  Dave Quah•660

  @MilleusPosted over 2 years ago

  Thanks for your feedback! I've indeed forgotten about that.

  Also, I've just found out that now major browsers implicitly sets rel="noopener" as default for <a> elements with target=_blank :o

  Dropping this here for future references:

  • https://chromestatus.com/feature/6140064063029248
  • https://developer.mozilla.org/en-US/docs/Web/HTML/Link_types/noopener

  0