All comments

• 

  FranekHincmann• 190

  @FranekHincmann

  Submitted about 1 month ago

  

  My Solution to this Challenge - QR code component

  1

  20

  

  Rebecca O'Neill• 30

  @rebeccaoneill

  Posted 22 days ago

  Hi FranekHincmann,

  Code and solution overall look great so nice job. One thought that might be useful for future: You could simplify your html by removing some of your divs, it's possible to include the two text elements in one div for example, this may mean you don't need to repeat some of the styling such as padding which may be consistent for other components.

  Happy coding :)

  Marked as helpful

  0