Design comparison
Community feedback
- @devchsykasPosted 5 months ago
Hi, nice work, i like your project but i have a few suggestions
-
Semantic HTML Elements: Consider using more semantic HTML elements to improve accessibility and SEO. For example, <main> could be used to wrap the main content area instead of <div class="card">.
-
Class Naming: Ensure consistent and meaningful class names. For instance, card_item1 and card_item2 could be more descriptive, such as qr-code-container and content-container.
-
Footer Attribution: Ensure the footer attribution is easily readable and accessible. Consider increasing the font size (font-size: 11px; may be too small for some users).
Marked as helpful0 -
- @eseoxaldePosted 5 months ago
Great job. Maybe you should separate the CSS from the HTML by creating a new file
0
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