@santiagosg
Posted
Hi Anna
I think you use too much the div for elements that don't need it, like in the plan (the image and the plan-change).
You are using a button to "proceed payment" and "cancel order", that's good and its ok, but remember to add a "cursor: pointer;"
Try to organize your documents, like all the images in the images folder, and maybe your style.css under a css folder too.
You are doing great, keep moving.
P.D. I liked your way to show the screenshot in GitHub
Marked as helpful
@akarinaa
Posted
@santiagosg thank you very much for your insight, Santiago! It's really helpful. I'm learning ways to make my code/project cleaner, so your advice is very appreciated.
I chose a mockup to showcase the design because I'm really like how the mobile version turned out, so I wanted to show it along with the desktop version. I might do this for my next projects as well.
