QR Code Component Using Custom CSS Variables and CSS Grid Properties

Community feedback

• 

  LeikoRed•10

  @LeikoRed

  Posted 10 months ago

  I see you are using:

  not(ul, ol, li) {margin: 0; padding: 0;}
  

  since it targets all elements except ul, ol, and li, setting their margin and padding to zero. It'd be prefered to use the universal selector. The :not selector is used for more specific cases where you need to exclude particular elements, but in this scenario isn't necessary because you don't use ul, ol or li.

  *{
  margin: 0;
  padding: 0;
  }
  

  It is good tho

  01
• 

  Elian Pujols•30

  @ElianPujols

  Posted 10 months ago

  Espero que tengas esto en cuenta. La solucion esta correcta, pero un punto en el codigo es la cantidad de comentarios repetidos que tiene, en realidad no interfiere directo con el codigo pero es algo que a mi opinión se debe de utilizar en la menor medida posible...

  02