Skip to content
OverviewSolutions
9
Comments
9

Lara Mesa

@Lara-artCanary Islands, Spain250 points

❤ Amor por el Diseño web y el modelado 3D 🖥 Junior Frontend Developer. React.

Latest solutions

Latest comments

  • Lara Mesa•250
    @Lara-artPosted about 1 month ago

    Hey! 🙌
    Your work is impeccable. I love that you did everything with React and Vite because it makes it reusable in other parts of the website. The only downside is that you didn’t include where you deployed it in the README, so it can’t be tested. Make sure to add it so potential clients can try it out.

    Great job, congratulations! 🎉

    Marked as helpful
    1
  • Lara Mesa•250
    @Lara-artPosted about 1 month ago

    Hi! 🙌

    It's a good solution, but there's a problem. When you click the .share button, all the content shifts. After checking, I noticed it's because the .social-media div is outside the footer. If you move it inside, the content won't shift, but you'd also need to adjust the CSS a bit:

      top: -70%;
  left: -20%;
  display: none;

    And remove overflow: hidden; because it cuts off the .social-media div if you want to move it inside the footer.

    And @media doesn't works

    0
  • Konrad•370
    @konradbaczykSubmitted 8 months ago
    What challenges did you encounter, and how did you overcome them?

    The biggest challenge was positioning cards with only grid on different screen sizes. I wanted to learn more using grid so I read some MDN articles to better understand how to use grid. Finally cards are on the right positions :)

    Testimonials-grid-section

    1
    Lara Mesa•250
    @Lara-artPosted 8 months ago

    Heeey! 👋 Yeah, grid can be tricky at first, but I think you've done a great job.

    The way you did it is perfect, and you could have also solved it using grid-template-areas, which might seem a bit odd at first. Here’s a link if you want to check it out: https://developer.mozilla.org/en-US/docs/Web/CSS/grid-template-areas.

    Keep it up, your solution is awesome!

    1
  • Lara Mesa•250
    @Lara-artPosted 8 months ago

    Heyyy! 👋 Your solution looks a lot like the one we were given, congrats!

    However, I noticed the images aren't showing up because the URL is incorrect. Since they're in the same parent directory, you just need to add a "/", so it should be <img src="/images/icon-supervisor.svg" alt="">.

    Everything else looks great. I did the exercise differently, but I find your approach easier.

    Marked as helpful
    0
  • Lara Mesa•250
    @Lara-artPosted 8 months ago

    Hey there 👋I was checking out your GitHub and it looks awesome. Your way of structuring the code is very readable, it looks a lot like the original. The only mistake I noticed is that you didn't use class="fa-solid" in the CSS, but it's no big deal. Great job!

    Marked as helpful
    0
  • Raj Kumar Mannam•100
    @RajKumar-612Submitted 9 months ago
    What are you most proud of, and what would you do differently next time?

    Did most of the task on my own, but took help on some of the styling.

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

    css challenges on the table,list and some alignments, took help of chatgpt and online resources

    Recipe-page

    1
    Lara Mesa•250
    @Lara-artPosted 9 months ago

    Hi! 👋First of all, congratulations on completing the exercise. I checked your GitHub and wanted to mention a couple of things: use the <section> tag to separate each of the blocks, even if you later use <div>, because this helps other developers understand your HTML better. Also, the image appears stretched—try using: img { width: 100%; } It will look better. Good luck with the next project! ❤

    Marked as helpful
    0
View more comments