Design comparison
SolutionDesign
Community feedback
- @JeyB88Posted 4 months ago
great job! some small improvements I would made:
- In my opinion h3 or h2 is from the semantic perspective a better choice as h1 for the qr code title
- What if there is content to add to this component? Right now the height is fix to 497px and will hide the new content. I personally only specify the width and use the auto height and let the spacing between the elements do the rest
Marked as helpful1
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord