Skip to content
OverviewSolutions
4
Comments
43

Samuel Palacios

@samuelpalaciosdevCaracas, Venezuela615 points

Frontend developer who loves code. Building tools for humans.

Latest solutions

Latest comments

  • Samuel Palacios•615
    @samuelpalaciosdev

    Hello, Sirriah 👋

    Well done on this challenge!. Your solution looks nice (almost pixel perfect) and it scales pretty well.

    I only suggest some things😉:

    • Adding outline: none to the buttons with the icons. When I click on them, that outline doesn't look bad but I'd not display it.

    • About your sass stylesheets. I suggest you to watch [this video] (https://www.youtube.com/watch?v=9Ld-aOKsEDk&ab_channel=KevinPowell)

    I really like the way you approach this, keep coding ;).

  • Samuel Palacios•615
    @samuelpalaciosdev

    Hello, William! 👋

    Great job on this challenge!. Your solution looks nice (almost pixel perfect) and it scales pretty well.

    I only suggest some things😉:

    • Adding cursor: pointer to your button. So people would know that's clickable.

    • Take care of those accesibility issues. I suggest you to use the [W3C Markup Validation Service] (https://validator.w3.org/).

    I really like the way you approach this, keep coding ;)

  • Samuel Palacios•615
    @samuelpalaciosdev

    Hello, Spymom! 👋

    Well done on this challenge!. Your solution looks great and it scales pretty well.

    I only suggest a little thing😉:

    • I think that those images, don't need an alt text. In this instance, that's a good thing, as the alt text is currently repeating the same as the card headings, so if you take this tip, use alt="" aria-hidden="true" on your HTML markup.

    I really like the way you approach this, and that fancy animations you added are so cool, keep coding ;)

  • Samuel Palacios•615
    @samuelpalaciosdev

    Hi, ApplePieGiraffe👋

    Congratulations, another amazing solution from you (as expected). I love the soft and cool animations you used. Your work impresses me, again, congratulations, mate ;).

    Keep coding🧡

  • Samuel Palacios•615
    @samuelpalaciosdev

    Hi, Orlando👋

    Well done on this challenge!👍

    I only suggest some things 😉:

    • Be clear with your HTML classes. I don't understand what you're trying to say with example: flav or indi.

    • Using min-widths and min-heights instead explicit width a height properties. Setting explicit widths and heights usually is not the best approach, As I'm seeing on your project, on the 900px viewport, one of the containers (the right one) lacks information, if you use min-widths, it will help you.

    I hope this would help you, have a nice day, keep coding!💙

  • Samuel Palacios•615
    @samuelpalaciosdev

    Hi, Sven👋

    Good job on this challenge! Your solution looks good and it scales pretty well.👍

    I only suggest some things 😉:

    • Changing the styles for the social media icons. As I see on you project, the twitter icon looks different from the others. That's because you set a width on them, I'd delete it, makes them look better.

    • Setting a padding on the main element. You only have padding for the left & right, but the "Social media dashboard" is too close to the top of the page, I'd add padding: 2rem 3rem on the 768px media query.

    I hope this would help you, have a nice day, keep coding!💙

View more comments