Skip to content
OverviewSolutions
18
Comments
6
P

kilozdazolik

@kilozdazolik190 points

🤡

Latest solutions

View more solutions

Latest comments

  • P
    kilozdazolik•190
    @kilozdazolikPosted about 2 months ago

    the colors look different other than that its good!

    0
  • P
    kilozdazolik•190
    @kilozdazolikPosted about 2 months ago

    good

    0
  • mauriceprosper•40
    @mauriceprosperSubmitted 2 months ago
    What are you most proud of, and what would you do differently next time?

    Built a full-page, as regards to regular components.

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

    I didn't encounter any challenge!

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

    No where!

    Recipe Page

    1
    P
    kilozdazolik•190
    @kilozdazolikPosted about 2 months ago

    The solution does not meet the design. Try fixing the colors, sizes, and styling.

    0
  • P
    kilozdazolik•190
    @kilozdazolikPosted about 2 months ago

    the solution looks bigger. other than that good job.

    Marked as helpful
    0
  • P
    kilozdazolik•190
    @kilozdazolikPosted about 2 months ago

    great job

    0
  • P
    kilozdazolik•190
    @kilozdazolikPosted about 2 months ago

    Use a <figure> and <figcaption> for the QR code image since it's a meaningful piece of content.

    The text is crucial information, so using a heading (<h1> or <h2>) for the main title would be better for accessibility. Add alt="QR code linking to Frontend Mentor" to clarify what the QR code does.

    Instead of padding: 16px; in .improve, use margin-bottom to separate elements more naturally.

    Add cursor: pointer; to the QR code image for better UX.

    Other than that it looks great! Awesome job!

    0