Skip to content
OverviewSolutions
10
Comments
8

Patrick Cuentas Mariano

@PatrickCuentasPerú260 points

Hey you follow me on github 😎

I’m currently learning...

Angular and NestJS

Latest solutions

View more solutions

Latest comments

  • Patrick Cuentas Mariano•260
    @PatrickCuentas

    Good Work, Pixel Perfect style. Keep up coding. Happy coding :)

  • Patrick Cuentas Mariano•260
    @PatrickCuentas

    I think it's better to use REM units that are based on the font size of the root element.

    Mainly for accessibility issues since users have different default sizes other than 16px for different reasons.

    And when using REM units, the size is based on that default size of each user, which offers a better experience.

    I am not an expert, but I hope you can use my suggestion for a broader search on the subject.

    Greetings ✨

    Marked as helpful
  • Patrick Cuentas Mariano•260
    @PatrickCuentas

    Good job @brunafields.

    Recommendation:

    • Change the color of the text so that it contrasts more with the background of the card so as not to strain the eyes as much.
  • Patrick Cuentas Mariano•260
    @PatrickCuentas

    Good job, I would suggest you add javascript to give it life or better learn a framework when you feel with a good base of javascript and be able to handle the requirements that they ask for faster.

    I invite you to see my solution to the same challenge made with ReactJS.

  • Patrick Cuentas Mariano•260
    @PatrickCuentas

    Hi, I'll help you with a snippet so you can improve the centering of your design in tablet+ resolution.

    @media screen and (min-width:992px){
    body {
         height: 100vh; 
        display: flex;
        flex-direction: column; 
        justify-content: center;
        align-items: center;
    }
}
  • Patrick Cuentas Mariano•260
    @PatrickCuentas

    Seems you need to zoom fit 150% to look great, good job anyways.

View more comments