Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refine Rolodex Radar Observer Configs #5

Open
israelias opened this issue Sep 21, 2021 · 0 comments
Open

Refine Rolodex Radar Observer Configs #5

israelias opened this issue Sep 21, 2021 · 0 comments
Labels
bug Something isn't working

Comments

@israelias
Copy link
Owner

The responsive nivo Radar animates to the data in batches of 25 pokemons in view as you scroll through the Pokedex (rolodex). This works logically in development, with minor threshold adjustments in the observer, but is out of sync in production.

My hunch is that

  • Radar needs more fragmented side-effects to react to independently.
  • Do not rely on serverSideProps for the query data, although graphql-codegen-apollo-next-ssr was a game-changer in getting hooks to function on a pre-render outside of a function component.
  • Conversely, SSR is surely adding to the sync problem. CRA and/or divorcing the notion of pages from components could set the pokedex free
  • NextJS does not play well with ContextAPI as the tendency is to refresh the page between links unless NextLink wrapper is used, which is not easy to test in a hot-reload dev environment

On a side note, routing to pokemons/[name] is meant to capture that pokemon's segment of the radar. Only the route changes. Locally, this is relatively working as it freezes the radar svg's state at the moment of routing.

@israelias israelias added the bug Something isn't working label Sep 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant