QR code component with HTML and CSS

@Alex-Archer-I

Thank you so much for your feedback!

You’re right, using rem values would be more appropriate, especially for accessibility. And I agree, I did overuse the font-family declaration. 😅

I’ll update my project with these changes, as they seem very relevant. Thanks again! 🙂

PS: For the emojis in commit messages, I use the Gitmoji convention, which I find helpful for quickly understanding the type of changes made.

@HLIFT

Hey, you're welcome! 😄 Your comments about github (both of them 🙂) are very cool and helpful too =)

@dev-italoregis

Thank you so much for your feedback!

To organize your repo, I recommend structuring your files into specific folders rather than placing everything at the root. For example, all images should go into an ‘images’ folder, and if you have multiple style sheets, they could be grouped in a ‘css’ folder. This helps make your repository more readable and makes it easier to understand where everything is located. Additionally, it simplifies finding specific files. If you’re using frameworks, they typically come with an organized structure that you can follow 🙂.

Regarding commits, it’s important that each commit focuses on a single task. For example, you shouldn’t group multiple new features into a single commit. And most importantly, commit messages should be clear, so that the nature of the changes is immediately understandable. For instance, if you’re fixing a typo, your message should be something like ‘Fix typo’ rather than just ‘Fix’. Personally, I also use the Gitmoji convention, which I find helpful for quickly understanding what was done and adds a bit of visual flair to commits. But that’s just a bonus, not a necessity—the first two points are the most important.

As for your feedback, you’re absolutely right. I’ll update my code to implement semantic HTML, as it’s definitely better for accessibility! Thanks again! 🙂