-
Notifications
You must be signed in to change notification settings - Fork 3
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
fix(bouquet): addresse le loader infini #171
Conversation
✅ Deploy Preview for ecospheres ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice!
Je pense qu'il faudrait utiliser les deux composants dans du "vrai" code existant avant d'introduire le changement, sinon c'est juste du dead code en attendant.
Also, rebase à faire? Il y a des changements sur la conf typescript qui n'ont rien à faire là je crois?
Hey ! Oui c'est marqué la #170 comme dépendance pour la config ts. |
Changer la base branch dans ce cas :-) |
+1, ça facilitera la compréhension de la PR aussi, et accessoirement se rendre compte si ça fonctionne bien comme on le souhaite :) Pour le "vrai code", pas de préférence entre ajouter dans cette PR ou préparer une autre PR basée sur celle là. |
b9c14bd
to
cc3dfdd
Compare
Depends on #170
Fixes #167
Fix
Contexte
Quand-on attend qu'un page charge, on est habitué à ce que l'on puisse annuler d'une manière la charge (si elle trop lente, par exemple) et continuer à naviguer sur le site.
Problem encountered by users
Aujourd'hui, si l'on change de page, il y a un spinner qui ne s'arrête jamais.
Proposal of how to solve the problem
Améliore l'expérience de l'usager en cas d'un loading trop lent.
What changes
Technical changes