I used HTML and CSS (flex)

👋Hi Marko!

👍 Good job on completing your first challenge on Frontend Mentor. I hope that you learn a lot when you built this project 😉. To be honest, the result still needs a lot of improvement.

So, I will give you some feedback:

• It's great that you wrapped all contents inside the landmark (main, and footer tag). 👍 Well done on this one.
• In this case the illustration and the music icon is decorative images. Remember, all decorative images, you should leave the alt="" empty and add aria-hidden="true" or role="presentation". That way the screen reader will ignore those images.
• You don't need to wrap the Change with p tag, an anchor tag itself is enough.
• Heading tags must go in order. You need to use h2 before you can use h3.
• The Proceed Payment and Cancel Order should be a button or anchor link (choose one).
• You should not defined the width on the main element. It should be a max-width so it doesn't become so large on every screen size.
• Use rem or sometimes em, don't use px. Pixel unit will not allow the user to control the size of your page.

Tip on writing HTML is that, imagine that you write it as a document (without any CSS), that way you can make a better decision on using the correct HTML tags.

That's it! Hopefully this is helpful and good luck with your coding journey!

Great Work On Your First Challenge Marko,

1. I would sugguest you use Figma/adobe XD for mapping the size of elements so that you would get the exact result as of design, your design is quite big compared to the og design.

2. Mobile responsiveness is missing.

3. Increase the spread and blur of box-shadow and reduce the alpha of RGBA, something like this: box-shadow: 10px 10px 51px 14px rgba(0,0,0,0.24);

you can use the tool like these to get your box-shadow right:

https://cssgenerator.org/box-shadow-css-generator.html

https://getcssscan.com/css-box-shadow-examples

I Hope These Feedback Helps You :]

For learning only 1 week of html and css really good. I'm learing from 3 months still not good enough. Here if you can remove the box -shadow and also see the colors provided by front-end-mentor style.md i belive, but overall really nice.