Alieu• 190
@alieuk61
Submitted
I enjoyed making this and think I did pretty well on it, however I'm always open to any feedback and will appreciate if I'm given any :)
Alieu
@alieuk61
All comments
- Alieu• 190
@alieuk61
Posted
Also I just realised that I made a few mistakes with the colours of some things...
0 - Alieu• 190
@alieuk61
Submitted
I didn't use js because I don't know how to use it, I'll come back to this project in the future to modify it, however in the mean time can someone please tell me how to make a hoverable drop down menu it would mean a lot. I feel like there was a problem with my html which also didn't allow the dropdown menu to work, so can someone give me feedback on that as well. Thanks :)
Alieu• 190@alieuk61
Posted
also can someone please let me know if the actual website looks different/the same to solution displayed in the design comparison, because to me the solution that i see looks worse than what i see when i click on the website.
0 - Murilo Pita• 60
@murilopita
Submitted
All feedback is welcome! Thank you! :D
Alieu• 190@alieuk61
Posted
looks great!
Marked as helpful
0 - mikosis123• 40
@mikosis123
Submitted
hi guys its my first challenge as junior front end developer on this cite.it was a bit challenging form me so pls let me know ur comments.
Alieu• 190@alieuk61
Posted
it looks very good, but I think maybe if you made the body have box-sizing: border box, it will allow you to add paddings to the body without allowing it to get bigger in width and height. also in my opinion I think its good practice to have the height have min-height: 100vh; and have the width have max-width: 100vw. Hope this helps and can you also give me some feedback on my page thanks :)
Marked as helpful
0 - Kate• 30
@unic0rnKate
Submitted
Please don't judge me hard, i am completely beginner, have some mercy ok? :) i found difficult to make a boarder-radius of my image and description div, because they will change their border-radius separately and not gonna look as one box with rounded corners.
Alieu• 190@alieuk61
Posted
It's ok, we're all trying to get better, your css isn't showing as Chris said however when you fix it I can come back and try to give some help :)
0 - Abdulquddus ibrahim• 150
@Xclusive09
Submitted
Open for corrections.
Alieu• 190@alieuk61
Posted
This looks so good, literally a replica of the design. Well done :)
0 - Alvaro Flores• 20
@alvaroflores01
Submitted
difficult? Hard time finding an easy way to vertical-align the prices without having to use flex box which seems overkill to be able to do that.
Alieu• 190@alieuk61
Posted
I put the prices in separate divs within a container div and then used margins to try and vertically align them, you can also use flex box to try and do that as well, I hope this helps :)
0 - Ariel Rubel• 70
@pepepinguino69
Submitted
Tips and advice always welcome
Alieu• 190@alieuk61
Posted
Desktop design looks great but the mobile design is supposed to be different, also can you please put the correct link to your code? because when I click it, it comes up with page not found. :)
1
