Skip to content
OverviewSolutions
6
Comments
5
P
Karen L Byrd
@klb-dev

All comments

  • Karen L Byrd•80
    @klb-devPosted 4 months ago

    Looks amazing very similar to the original design.

    0
  • Monica I.•200
    @monicaivanovSubmitted 4 months ago
    What are you most proud of, and what would you do differently next time?

    This was not a difficult project but it required a lot of change in the properties as the design had different details. Im proud I did find a solution to all and Im happy with the result. Hopefully as I keep learning I will improve my code solutions.

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

    I had to remind myself the list and tables properties (something I rarely use). w3school has saved me many times.

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

    Any suggestions on what I could have done better or simpler I will greatly appreciate. Thank you.

    Receipe main page

    2
    Karen L Byrd•80
    @klb-devPosted 4 months ago

    Your page looks very sleek. You followed most of the design, just some color of the content. You have black, but I believe the color is stone-600 (hsl(30, 10%, 34%). It just stands out in black but does not look horrible. Great job, keep it up.

    0
  • Karen L Byrd•80
    @klb-devPosted 4 months ago

    I love that you added your picture and location. The code looks sleek and works well.

    0
  • Karen L Byrd•80
    @klb-devPosted 4 months ago

    Great job on the design and coding so far. Your efforts are evident and appreciated. The code is clean, but why not use <button> or <a> for the learning button? You placed it in <p>. The heading should be an <h1> as well. These tags are done for screen readers and people with visibility issues. Your avatar image does not show up even though it looks correct. Placing a '.' before the first '/' would make it work.

    CSS looks good; maybe start with mobile first and then have fewer media queries to produce more DRY code—great job with styling.

    Marked as helpful
    0
  • Karen L Byrd•80
    @klb-devPosted 4 months ago

    NA

    0