@am4188
Hi! :) I'm a student studying Mobile Application and Web Development. I love learning new things and am always eager to expand my knowledge!
I’m currently learning...Mobile Application and Web Development. Focusing on CSS, HTML and Javascript. Want to branch into learning more frameworks and libraries. Also want to dip my toes into back-end.
Latest solutions
Latest comments
- @YayoKB
I've only glanced at this for a bit so forgive me if this isn't incredibly constructive or helpful. :)
- You're using fixed widths (pixels instead of rem / em or vh, vw etc). This decreases the levels of accessibility and responsiveness, too.
- I haven't come across using spans the way you did. Is there any reason you did so?
- For some reason, .preview does not follow the same widths as its parent. That is why your button has strange sizing - it's simply following .preview which is bigger than your .container. I could be wrong though - I'm just starting out on here too.
Marked as helpful - @dracowarz@YayoKB
- The site is not responsive. There are two layouts, one for desktop/wider screens and one for phones/taller screens. Your site does not change the layout depending upon the screen width. Please do that.
- Some of your colours do not match exactly. The price and button should be a different green and the description and "PERFUME" label at the top should be in grey.
Marked as helpful
