Skip to content
OverviewSolutions
8
Comments
8
Leonardo Meza
@leonardomeza87

All comments

  • Leonardo Meza•245
    @leonardomeza87Posted over 3 years ago

    Hello, even though your solution looks good, it generates a lot of lag when I'm on the main page, I suggest you look for a solution to that problem.

    Good job with the rest of the page 👍

    0
  • Leonardo Meza•245
    @leonardomeza87Posted over 3 years ago

    Try using max-width in pixels instead of percentage when you are no longer in mobile resolution

    Btw it's funny that we have the same last name xD

    1
  • Leonardo Meza•245
    @leonardomeza87Posted over 3 years ago

    Looks pretty good ^ _ ^

    As an improvement you can add a transition to the elements with hover

    Marked as helpful
    0
  • Leonardo Meza•245
    @leonardomeza87Posted over 3 years ago

    Hi, I am fine with the way you structured your files, good job.

    Testing a bit I found that I cannot change the filter by region without restarting it, that is, filter from Americas to Europe, countries are not loaded that way.

    Also, if I write a country code that does not exist in the address bar, it loads the page of a country without any data.

    0
  • Leonardo Meza•245
    @leonardomeza87Posted over 3 years ago

    Hola, en YouTube puedes encontrar este curso sobre flexbox que de seguro te ayudara: https://www.youtube.com/playlist?list=PLvq-jIkSeTUbQc3dGsssp8lxAi5npMrys

    Su canal es muy bueno, te lo recomiendo!

    0
  • Leonardo Meza•245
    @leonardomeza87Posted over 3 years ago

    Hello, your application works but the icons of the text fields are flying 🐛

    By the way, I know it can be a bit difficult to find the solution but I don't think it is the best to reload the page by clicking the reset button, use the benefits of React ⚛ as a single page application

    0
  • Leonardo Meza•245
    @leonardomeza87Posted over 3 years ago

    Greetings, your app works fine but it makes a strange animation when it loads the page and the transition from light to dark mode is a bit abrupt

    I recommend deactivating the animations for the elements that do not need it 👍

    0
  • Leonardo Meza•245
    @leonardomeza87Posted over 3 years ago

    To check the compatibility of the CSS styles you use you can visit the site: https://caniuse.com

    And it seems that your title lacks a little margin up on my phone 📱

    Marked as helpful
    0