@xeuxdev
Posted
Wonderful, but there's no need for the navigation 😊. Another thing is - "make use of relative units- Like rem " because it scales better on devices😊. Hope it helps😊
Marked as helpful
@Khadijarejjaoui99
Posted
@Headbwoi Thank you!
Submitted
@Khadijarejjaoui99
Hello, I appreciate any kind of feedback :)
@xeuxdev
Posted
Wonderful, but there's no need for the navigation 😊. Another thing is - "make use of relative units- Like rem " because it scales better on devices😊. Hope it helps😊
Marked as helpful
@Khadijarejjaoui99
Posted
@Headbwoi Thank you!
@GeorgeCaldarescu
Posted
Hi! Some advices, I'm not an expert but I start to write my HTML cleaner and your looks ok from my opinion. Try to start to work on the mobile version first, as you will see the divs will automatically arrange on columns, and then move on the desktop part you will have. I suggest you the Kevin Powell CSS course on responsive designs (is free), he's focusing more on the flexbox. If you like you can also have a look at the 2 courses of Wesbos regarding flexbox and grid.
The nav menu is a nice add, but you can use white color so can popup better, and on mobile version try to put the hamburger menu, even if is not working.
Cheers
Marked as helpful
@Khadijarejjaoui99
Posted
@GeorgeCaldarescu Thank you!
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