Hi, first of all website looks very good, everything is on point. In terms of simplicity html and css looks quiet good. looks like you have quite good understanding how things work...
But as you have already mentioned "div soup" - that is a aspect you need to work.
Few suggestions:
- fro card use semantic element <article></article>
- Maybe you also can use <main></main> instead of div.container, but not 100% necessary.
- Always use buttons for things like: "Proceed to Payment" and "cancel request", you game them classNames called "button" - why the hell don't you also gave them button element tags XD ?!! in "cencel order" -this looks like a link but semantically it is button, when user clicks it you have to cancel order, that is button functionality not link functionality... so ya, try to call it className btn-link or even link is OK, but use <button> instead of <div>
- for div.text, instead of div use <p>.
- in general try to watch few tutorial on youtube about semantic html and google thing, it will help you.
Marked as helpful
3
