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

All comments

  • Alexandru736β€’ 210

    @Alexandru736

    Submitted

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

    So far, I am proud of what I have achieved on this project(being able to consolidate my CSS knowledge and skills). Depending on the feedback I receive, I might do different things.

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

    I didn't know at first how to use the table HTML element and how to style it. I had small difficulties with adding padding only to the first column because the other column would always shift with the same padding distance applied to the first column. After researching, I discovered that you could set the max width in percentage to each column. In my case the full width would only be 50% of the available row width, regardless of what padding I would apply to the first column.

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

    I don't know if the way I styled the whole project is the most efficient. I know everybody has his one way to style a html page, but there might be some best practices I didn't follow right. I would appreciate if you could point out what areas I could have improved or made them better.

    Henrique Cruzβ€’ 130

    @Henr1queCruz

    Posted

    Hello there! πŸ‘‹First of all, congratulations on completing this challenge!

    Your solution looks great!

    I have a suggestion for improvement:

    Reagarding Semantic HTML, I would suggest you to use the <section> tag instead of using <div>s.

    Another observation I've made is that you're not utilizing the class for the <hr> tag <hr class="solid">. You might want to consider using the <hr class="divider"> that you're already styling, or alternatively, you could remove the <hr> tags and its corresponding styling, from your code altogether. Instead you could add border-bottom property to the appropriate classes such as ingredients-container and instructions-container.

    Keep up the great work! 😊

    Marked as helpful

    0
  • P
    CHBNDJβ€’ 390

    @CHBNDJ

    Submitted

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

    NONE

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

    NONE

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

    NONE

    Henrique Cruzβ€’ 130

    @Henr1queCruz

    Posted

    Hi there πŸ‘‹. Amazing work!

    I suggest considering the use of flexbox or grid to enhance the responsiveness of the page. Relying solely on position: absolute with specific pixel values for positioning elements can result in a static layout.

    Additionally, there appears to be a misspelling in the class name of the last links group.

    Lastly, incorporating pseudo-classes such as :hover could enrich the user experience by providing interactive feedback when users interact with the links.

    Continue the great work! 😊

    Marked as helpful

    0
  • Henrique Cruzβ€’ 130

    @Henr1queCruz

    Posted

    Hello there πŸ‘‹. Good work overall!

    Consider removing one div that is being overused and using the <main> semantic tag to improve code readability and accessibility.

    Keep it up!

    Marked as helpful

    1
  • sridhar0612β€’ 60

    @sridhar0612

    Submitted

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

    good

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

    time

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

    time

    Henrique Cruzβ€’ 130

    @Henr1queCruz

    Posted

    You can slightly enhance the card by making it wider to fit more of the original design. One last thing to improve accessibility is to add the alt attribute to the image. Great work!

    0