Samuel Palacios

@samuelpalaciosdev
Caracas, Venezuela
615Points

Frontend developer who loves code. Building tools for humans.

Latest Solutions

Latest Comments

    • HTML
    • CSS

    Fylo data storage / sass, mobile first, BEM

    2
    Samuel Palacios615 | Posted 6 months agocommented on Tereza's "Fylo data storage component" solution

    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 ;).

    0
    • HTML
    • CSS
    • JS

    Ping Coming Soon Page - HTML & CSS

    5
    Samuel Palacios615 | Posted 6 months agocommented on William Luft's "Ping single column coming soon page" solution

    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 ;)

    0
    • HTML
    • CSS

    Four card feature section | Sass, BEM, GRID and Tilt.js

    2
    Samuel Palacios615 | Posted 6 months agocommented on spymon's "Four card feature section" solution

    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 ;)

    1
    • HTML
    • CSS

    huddle Landing page using Flexbox

    6
    Samuel Palacios615 | Posted 6 months agocommented on Chris Eden's "Huddle landing page with a single introductory section" solution

    Hi, Chris👋

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

    I only suggest some things 😉:

    • Changing the button to an <a> (anchor link). If this would be an actual webpage, those buttons should redirect somewhere.

    • As Karim said, instead of using overflow:hidden you should use overflow-x:hidden. Because, if you left as it is right now, I can't scroll on mobile viewport.

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

    1