Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

HTML/ CSS ( grid / flexbox/ @media ) a new proprieties

@jonathanPons

Desktop design screenshot for the Recipe page coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


What are you most proud of, and what would you do differently next time?

I improved my CSS, delved deeper into and lists, and completed the challenge in 2 days (around 8 hours of work from start to finish). I would like to rely less on ChatGPT and save time on properties to have cleaner code.

What challenges did you encounter, and how did you overcome them?

I struggled to change the colors of the bullet points and numbers on the lists, and I sought help on the internet and unfortunately on ChatGPT 3.5.

What specific areas of your project would you like help with?

Improve my CSS, my execution speed, and try to have cleaner and more efficient code. I'm not sure if 8 hours is a lot for a project like this for a beginner?

Community feedback

@DylandeBruijn

Posted

@jonathanPons

Hiya! 👋

Congratulations on your solution, it looks very close to the design! I can tell you put a lot of effort into it.

Things I like about your solution 🎉

  • Use of semantic HTML elements

Things you could improve ✍️

  • Try experimenting with the CSS layout tool Flexbox, it will help you greatly structuring elements on your webpage.

  • You could add a min-height: 100vh to your body element so it takes up the full height of the viewport while still being able to grow when the content inside it grows.

  • Try experimenting with CSS variables, they help you make your CSS values more reusable across your code.

  • I suggest using clear descriptive CSS classes like .card, .card-title and .card-description.

  • By convention CSS classes are formatted like this container-main. It's not recommended to use capital letters in your classes.

  • Try to make your recipe more responsive, the padding at the top should disappear on smaller viewports.

I hope you find my feedback valuable, and I would appreciate it greatly if you could mark my comment as helpful if it was! 🌟

Let me know if you have more questions and I'll do my best to answer them. 🙋‍♂️

Happy coding! 😎

Marked as helpful

0
geomydas 1,060

@geomydas

Posted

Gotta get them googling skills. Use ::marker pseudoselector if you want to manipulate the bullet/numbering on lists. I don't reccomend using ::before for it, ChatGPT isn't always the best. You would use it like this

li::marker {
color: red}

8 hours is a too much time for a newbie project like this but keep on making newbie projects and soon it will get faster.

Marked as helpful

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

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