Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

simple html css

Wasu44 50

@Wasu44

Desktop design screenshot for the QR code component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


What specific areas of your project would you like help with?

What could I do potentialy do better in my future projects? How to improve my coding to be more professional?

Community feedback

@armson45

Posted

Hello there! I'm a newbie too but let me give you some advice.

You're html structure is pretty good but try to use the semantic html5 tags instead of divs to improve the accessibility and ceo. Answering to your question: -Use semantic html tags. -Having your styles in the same html file doesn't look good, and also, when it gets larger it will be kinda confusing. -Try to use BEM (Block__Element--Modifier) to name your classes, it's easier when you write your styles because of the naming the hard part is to find a good name :(

That's all from me. Have a nice code! :D

Marked as helpful

0

@KocakAli

Posted

Hello, I am not professional frontende developer. Maybe I can't give you advice at this point but you can check my solution code and you can read my solution comments. Maybe this give you some idea.

0

Please log in to post a comment

Log in with GitHub
Discord logo

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