Skip to content
OverviewSolutions
5
Comments
5
wanlucas
@wanlucas

All comments

  • wanlucas•230
    @wanlucasPosted over 2 years ago

    olá, Luciana, acredito que você seja Brasileira, então responderei em português.

    Eu encontrei um bug no responsivo em algumas resoluções, 676 x 505 por exemplo. A caixa espõe a borda de baixo, o que destrói o layout.

    Com a largura de 600px a caixas ficam muito apertadas, talvez fosse melhor usar o Media querry um pouco antes ou diminuir a fonte.

    São erros bem simples, você fez um ótimo trabalho, parabéns!

    Marked as helpful
    0
  • giorgi•140
    @Lomsk1Submitted over 2 years ago

    resolutions are 1300-1500px & 375px;

    if u have any advice, please tell me to fix it and improve my skills

    sass, scss, css, html, JS,

    2
    wanlucas•230
    @wanlucasPosted over 2 years ago

    hello, @Lomsk1! I have a few things to mention that could be tweaked.

    when your project is opened in the mobile version, the buttons are totally disorganized. Responsive in general is not working very well.

    the console is showing some numbers, I believe they came from some test you did.

    there are some accessibility issues that could easily be fixed.

    everything else is fine, congrats.

    Marked as helpful
    0
  • wanlucas•230
    @wanlucasPosted over 2 years ago

    helo, @devZibah! I have a few things to mention that you could improve:

    the figures inside the buttons are slightly cropped.

    the header is too big compared to the order.

    the gap between the bottoes is a bit exaggerated.

    I think adding small animations could make the project more friendly.

    you're doing great, just need to fix a few things, I recommend taking a look at my project to get some guidance on animations.

    0
  • wanlucas•230
    @wanlucasPosted over 2 years ago

    hello, Mohamad Krayem!

    It's pretty cool, I like the theme change animation. but there are a few things to fix:

    The selector is not well centered when theme 2 is active.

    It would also be important for the expression to appear on the screen in real time, but the number disappears when the expression is typed.

    Marked as helpful
    0
  • wanlucas•230
    @wanlucasPosted over 2 years ago

    hello @ DevZibah!

    The panel is complete only after choosing the period. It is necessary for the page to load completely soon after opening the site.

    Mouse events are also missing.

    About the responsivity, it's not fitting the screen perfectly, I recommend using the percentage metric to fix this.

    Marked as helpful
    0