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

fix(bouquet): addresse le loader infini #171

Merged
merged 11 commits into from
Nov 22, 2023
Merged

fix(bouquet): addresse le loader infini #171

merged 11 commits into from
Nov 22, 2023

Conversation

bonjourmauko
Copy link
Contributor

@bonjourmauko bonjourmauko commented Nov 19, 2023

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

  • Crée un composable avec un loader et d'options quality of life :
    • Le loader s'arrête quand on clique sur 'back'.
    • Le loader s'arrête aussi quand on presse 'Esc'.
  • Teste la UI avec happy-dom

@bonjourmauko bonjourmauko added the enhancement Améliorations et nouvelles fonctionnalités label Nov 19, 2023
@bonjourmauko bonjourmauko self-assigned this Nov 19, 2023
Copy link

netlify bot commented Nov 19, 2023

Deploy Preview for ecospheres ready!

Name Link
🔨 Latest commit cc3dfdd
🔍 Latest deploy log https://app.netlify.com/sites/ecospheres/deploys/655dd56b4aa3530008eeae20
😎 Deploy Preview https://deploy-preview-171--ecospheres.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@bonjourmauko bonjourmauko mentioned this pull request Nov 19, 2023
Copy link
Contributor

@abulte abulte left a 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?

@bonjourmauko
Copy link
Contributor Author

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.

@abulte
Copy link
Contributor

abulte commented Nov 20, 2023

Hey ! Oui c'est marqué la #170 comme dépendance pour la config ts.

Changer la base branch dans ce cas :-)

@streino
Copy link
Contributor

streino commented Nov 20, 2023

il faudrait utiliser les deux composants dans du "vrai" code existant avant d'introduire le changement

+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à.

@bonjourmauko bonjourmauko changed the base branch from main to build/type-checks November 20, 2023 15:21
Base automatically changed from build/type-checks to main November 22, 2023 09:15
@bonjourmauko bonjourmauko added bug Quelque chose ne fonctionne pas and removed enhancement Améliorations et nouvelles fonctionnalités labels Nov 22, 2023
@bonjourmauko
Copy link
Contributor Author

@streino @abulte j'ai réduit le scope et changé la description à fix de #167

@bonjourmauko bonjourmauko changed the title refactor(api): introduit hocs fix(bouquet): addresse le loader infini Nov 22, 2023
@bonjourmauko bonjourmauko merged commit a3f2656 into main Nov 22, 2023
4 checks passed
@bonjourmauko bonjourmauko deleted the refactor/hofs branch November 22, 2023 11:44
@bonjourmauko bonjourmauko added this to the V1 : Découvrabilité par cas d’usage milestone Nov 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Quelque chose ne fonctionne pas
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Loader persistant lors d'un changement de page
3 participants