@JYLN
Posted
Great work on your solution!
I'm unable to review your exact code as it appears the GitHub link is broken, but based on the code I'm seeing through inspect, the code looks great!
One thing I did notice, which I'm not sure if it's a build issue (if you used React based on the tags of the your solution), but there appears to be a <br>
within the title (<h3>
) of the QR code card text. Ultimately, I believe you'd want to have this as one line of text and leverage CSS for the correct breaking of words. One easy way I can see this implemented in your code is by adding padding on the left and right side of your <div>
element containing the text. According to inspect, it appears there's only bottom padding.
Overall, great work!