@ogunsanwodavid
@cyeguez
Submitted
I've just completed a front-end coding challenge from @frontendmentor! 🎉
You can see my solution here: https://www.frontendmentor.io/solutions/responsive-landing-page-cssgrid-ewdjSc7sTT
Any suggestions on how I can improve are welcome!
@ogunsanwodavid
Posted
This is a job very well done considering you went mobile first For improvements, the h1 tags could have been made bigger (font-size) for mobile view but overall you aligned it perfectly.
@chungvuiloong
Submitted
Hi everyone,
Looking for feedback on my code so that I better code and make clean code.
@ogunsanwodavid
Posted
A job very well done. I don't need any other thing could have been added. A nice menubar idea too.
@prince-coder10
Submitted
How can I improve?
@ogunsanwodavid
Posted
You did a good job. But these are places you can improve
Marked as helpful
@victorrmark
Submitted
This is my solution for the Blogr Landing Page. Feedback is welcome
@ogunsanwodavid
Posted
Beautifully done. Commendable job.
@FazleLabib
Submitted
Would like to know what is the best way of creating the services section which consists of Graphic Design and Photography. I had a hard time positioning the text over the image to keep it consistent for different screen sizes.
Any other feedback regarding the whole design and suggestions on making the code more efficient is highly appreciated.
@ogunsanwodavid
Posted
Alright, The best way to achieve this is to create a div (wrapper) with the respective images as backgrounds and then create another div inside containing all the texts and give it margin-top:auto so they can be always positioned at the far bottom. You should also give the wrapper or container a padding so the texts dont sit right on the bottom of the wrapper. I hope this helps.
@JohanXTheKing
Submitted
@ogunsanwodavid
Posted
I suggest you use flex throughout. Don't even consider grid except for the slider/carousel maybe. Also, set height of major-sections of desktop-mode in vw and vh for easy-scaling.
