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
@alieuk61All comments
- @alieuk61Submitted over 1 year ago@alieuk61Posted over 1 year ago
Also I just realised that I made a few mistakes with the colours of some things...
0 - @alieuk61Submitted about 2 years ago
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 :)
@alieuk61Posted about 2 years agoalso 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 - @murilopitaSubmitted over 2 years ago
All feedback is welcome! Thank you! :D
- @mikosis123Submitted over 2 years ago
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.
@alieuk61Posted over 2 years agoit 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 helpful0 - @unic0rnKateSubmitted over 2 years ago
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.
@alieuk61Posted over 2 years agoIt'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 - @Xclusive09Submitted over 2 years ago
Open for corrections.
@alieuk61Posted over 2 years agoThis looks so good, literally a replica of the design. Well done :)
0 - @alvaroflores01Submitted over 2 years ago
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.
@alieuk61Posted over 2 years agoI 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 - @pepepinguino69Submitted over 2 years ago
Tips and advice always welcome
@alieuk61Posted over 2 years agoDesktop 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