Marcellmayy• 10
@Marcellmayy
Posted
Yes the solution is accessible and the code is well structured, readable and reusable.
0
Design comparison
Solution retrospective
What are you most proud of, and what would you do differently next time?
I am proud that i get to build a webpage and more importantly solve this challenge on building barcodes using basic html and css semantics
What challenges did you encounter, and how did you overcome them?
I had challenges with the styling sheet as I was confused about how to include either "%" or "px" to get the desired result of the preview webpage given. So, I had to check YouTube to learn a bit about what to use to be responsive.
I also am confused about the margins as well as the padding. I had to research and watch YouTube videos as well.
What specific areas of your project would you like help with?
The styling sheet should be looked and i will appreciate any feedback that will help improve the styling of the barcode webpage.
Community feedback
Svitlana Suslenkova• 3,490
@SvitlanaSuslenkova
Posted
<div class="qr-code-img"> you can add max-width here (it doesn't replace width)
body { display: flex; flex-direction: column; justify-content: center; align-items: center; min-height: 100vh; } Try this to align(top-bottom) and justify(left-right) your project to the center. It applies to the parent component(body), don't forget about !!min-height!!. You can use grid instead of flex too.
Hope you found this comment helpful :)
0
Marcellmayy• 10
@Marcellmayy
Posted
@SvitlanaSuslenkova Thank you very much for this feedback, I am really grateful for the pointers on the styling sheet. I will modify my codes to be better as advised. Thanks
0
Please log in to post a comment
Log in with GitHub
Join 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