Rimtar10• 60
@Rimtar10
Posted
hello!! so overall your project is really good keep on the hard work and let's improve together!! but i think that you could've aligned the text more at the middle of the page. And that's it, keep going!
1
Design comparison
Solution retrospective
What are you most proud of, and what would you do differently next time?
I'm proud of successfully completing the QR code component challenge in HTML and CSS. Next time, I aim to improve efficiency by incorporating more advanced styling techniques and ensuring cross-browser compatibility
What challenges did you encounter, and how did you overcome them?
I encountered challenges with aligning elements and ensuring responsiveness. I overcame them by utilizing media query for layout and testing across various screen sizes
What specific areas of your project would you like help with?
I'm eager to learn and improve. Do you have any suggestions on how I can make my learning process more effective?
Community feedback
Portal681• 30
@EkoNdongAyecaba
Posted
Hi @NivedhanaS ,
I just wanna ad that your id="desc" is not black, it has another color
I hope this suggestion will help you.
Stay working
0
Harsh Kumar Dwivedi• 740
@Harsh-Kumar-Dwivedi
Posted
Hi @NivedhanaS ,
I think you can prefer to replace <div id="info"> with <main> element and <p id="desc"> with <h1> element. These will improve accessibility.
You may use Flexbox for centering the card as it would be easier to center it with Flexbox.
Hope these suggestions are helpful !
Stay Happy & Healthy !
Happy Coding !
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