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

use of simple html tables, div containers for each section, basic css

PastaSus 80

@PastaSus

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?

First look accuracy of site on desktop

keypoints: Learn from others codes , add more code as much as possible for better accuracy.

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

css box property application , first time coding again in a while after learning fundamentals and the basics

consistency, relearn concepts through yt tutorials, and some googling

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

Site optimization on other devices better html code advanced styling on css

Community feedback

@Thakeeb22

Posted

You've written a nice code keep it but work on resizing the image then work on writing a more clean code try formatting your code using extension like prettier for vscode Wish us good luck

1
Samir 260

@Samir22Raj

Posted

Your site looks good at first view but as you take a closer look there are a lot of places that you botched. But well, we all gotta start somewhere right. My suggestion would be that you look into spacing in CSS. You don't have to exclusively define the width, margin, padding and all for each element. Use margin 0 auto to position your wholecontent and just let your browser take care of center positioning your div. Your image also remains static sized as you have defined its width. Images grow and shrink acc to the viewport so if you were aiming for the above design better to leave it at width: 100%. The table at the end too can be left as is in default. It will become responsive thanks to the browser. text-align: center is all you need, no paddings required. If you want to focus on reusability add CSS variables to your code.

Would also like to point you towards making a mobile design first so that you don't have to work too hard on the desktop version later on.

1

@ad-monir2001

Posted

Hey there, You have done a nice project. Keep coding. I wish a better future of you of coding.

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