Skip to content
OverviewSolutions
23
Comments
3
P
Jared Silver
@jsilver88

All comments

  • Jared Silver•500
    @jsilver88Posted over 2 years ago

    Nice job. I would reduce the loading times between each random piece of advice. Right now it seems a bit delayed. But overall it looks great!

    0
  • Chamu•13,130
    @ChamuMutezvaSubmitted almost 3 years ago

    Greetings to all. The journey continues with this exciting challenge. Navigating to the next page and moving focus to the top was something that i had to relearn from my previous projects - some of those things that you need to keep on doing before they stick in. There is still some adjustments that i have to do especially with the active states on the pricing page - the card has to move slightly to the top. It was easier if i had made the card active as an anchor element but the link button is a grandchild of the parent card. Your feedback is welcome.

    Photosnap website - using react and react-router v6

    3
    Jared Silver•500
    @jsilver88Posted almost 3 years ago

    Love the site!

    1
  • Jared Silver•500
    @jsilver88Posted almost 3 years ago

    Nice job. For semantics, I would wrap the "main-container" in a <main> tag instead of a div.

    Marked as helpful
    1