Skip to content
OverviewSolutions
9
Comments
38
Anthony Rosman
@D3press3dd

All comments

  • Anthony Rosman•510
    @D3press3dd

    Nice job Ana, really good job, keep it up!. you can add a little transition in the button hover and it give you more fancy details. let me create a pull request with the changes to show you

    Marked as helpful
  • Anthony Rosman•510
    @D3press3dd

    wow, you've grown quite a bit in the area, keep it up, check out this tool for animations https://www.framer.com/motion/ you can see what you can do in my solution to this challenge, it is really intuitive you can check this video https://www.youtube.com/watch?v=u_95SPKE6vg and see how it works in just 5 minutes

  • Anthony Rosman•510
    @D3press3dd

    Amazing job, looks really cool, keep the hard work and happy coding

  • Anthony Rosman•510
    @D3press3dd

    Hi Samson let me help you a bit

    1.-to get the color like the design you just need to use mix blend mode

    2.-you dont need to use a breakpoint like the style guide, they put that as a reference, so the idea is to do it completely responsive where the max screen is 1440 and the min is 375, thats mean that you dont have to worry to do the design for 4k screen or bigger than 1440

  • Anthony Rosman•510
    @D3press3dd

    Hola Camila, buen trabajo, lo unico que te podria recomendar es echarle un vistazo a los problemas de accesibilidad, el que dice que deben contener main element, es envolver la card en un main , agregarle un aria-role a los div, algo que tambien te recomendaria es hacer un reset a todos los elementos del documento para que no te afecte el box-sizing

    *,
*::after,
*::before {
  margin:0;
 padding:0;
 box-sizing:border-box;
}
    Marked as helpful
  • Anthony Rosman•510
    @D3press3dd

    I dont know why im getting the screenshot bugged like that 😢😢😢

  • Anthony Rosman•510
    @D3press3dd

    I hope you like it the fancy button animation in the dice

  • Anthony Rosman•510
    @D3press3dd

    Nice job Débora, if you want to add the local storage thing, its really easy, take a look to this https://developer.mozilla.org/es/docs/Web/API/Window/localStorage and if you need some help to apply it , let me know, happy coding

  • Anthony Rosman•510
    @D3press3dd

    I hope you understood my explanation because my english is really bad, but i try 😁😁, happy coding.

    Marked as helpful
  • Anthony Rosman•510
    @D3press3dd

    Take a look to this article Flex, if you use flex shrink to 0 it will overflow the parent, in this case the parent is main, so because yo have overflow:hidden the part overflowing its not showing

    Marked as helpful
  • Anthony Rosman•510
    @D3press3dd

    however when the screen width is less than 260px the paragraph text doesn't wrap this problem is because of the flex shrink flex shrink

    Marked as helpful
  • Anthony Rosman•510
    @D3press3dd

    Nice job, just fix the accesibility issues, just add alt text " " empty atleast, replace the div attribution for a footer for the lang error just add a main container and i think thats it, if you want to check the accesibility problems before upload your project, you can search on google chrome extension de axe accesibility its a good extension to check this kind of problem, happy coding.

    Marked as helpful
  • Anthony Rosman•510
    @D3press3dd

    Hi Juan really good job, i just would recommentd reduce some padding at top and bottom like this Padding

    Marked as helpful
  • Anthony Rosman•510
    @D3press3dd

    Hola Raul te quedo cool el diseño solo añadele un margin-top a la version desktop porque cubre toda la pantalla como puedes ver aqui preview

  • Anthony Rosman•510
    @D3press3dd

    Amazing job, just dont forget the accessibility issues, but nice work i like how you did it with react, amazing, happy coding💻💻

  • Anthony Rosman•510
    @D3press3dd

    Hi, just a few things -Dont use a static 85vh to the height because if i have a 4k screen, it will looks really weird, like this Preview

    -remember that if you add flexor grid the first level of childrens well be affect, the level 2 children not, so youre aplying flex to the container instead of the div cardfor that reason the content overflow it, look Preview

    -remember that you need to add the desktop version , if youre a little bit confuse maybe check some solutions on the community, you can check my solution too and thats it

    happy coding, if you have doubts, you can write me without problem💞💞

    Marked as helpful
  • Anthony Rosman•510
    @D3press3dd

    Amazing job, for fix the accesibility issues just add a main container and add a role to the divs happy coding if you dont know how to do it you can download an extension called axe accesibility

    Marked as helpful
  • Anthony Rosman•510
    @D3press3dd

    Amazing job, for fix the accesibility issues just add a main container and add a role to the div card, happy coding

    Marked as helpful