Skip to content
OverviewSolutions
16
Comments
6

kubas33

@kubas33180 points

I’m a mysterious individual who has yet to fill out my bio. One thing’s for certain: I love writing front-end code!

Latest solutions

View more solutions

Latest comments

  • kubas33•180
    @kubas33

    Input validation is broken. I can freely input month higher than 12, year higher than 20, 31 days in february etc

  • kubas33•180
    @kubas33

    Very nice design!

    But I found some bug.

    When you raise items to be added to cart and then click "Add to cart" then it will be possible to change items in cart directly on main page (by +/- buttons).

    I think you should be allowed to change number of items into cart only if you have cart opened and change value inside it.

  • kubas33•180
    @kubas33

    Very nice job! Although on mobile resolution the footer is little long (but I know that is needed in challenge design)

  • kubas33•180
    @kubas33

    Overall your solution is very nice but unofrtunately it's not fully responsible - it adds horizontal bar on lower desktop widths.

  • kubas33•180
    @kubas33

    I will not focus on code itself because I am beginner but I have some feedback according to UI.

    I think it's more intuitive that on Dark Mode version of site there will be Light Mode button and vice versa. Not Dark Mode button od Dark Mode version of site.

    Additionaly it would be nice if we can go back to main site when we chose any country.

  • kubas33•180
    @kubas33

    Hi! Thanks to your suggestion, appreciate it.

    But can you tell me why you suggest to wrap content with article? I am trying to understand this so I know when to use this or not. I thought it isn't necessery to wrap component into another tag if there is only one section.