Skip to content

[Prolongations] Affichage du _spinner_ sur la totalité du formulaire#6028

Merged
celine-m-s merged 1 commit into
masterfrom
celinems/prolongation-fixes
Apr 25, 2025
Merged

[Prolongations] Affichage du _spinner_ sur la totalité du formulaire#6028
celine-m-s merged 1 commit into
masterfrom
celinems/prolongation-fixes

Conversation

@celine-m-s

@celine-m-s celine-m-s commented Apr 24, 2025

Copy link
Copy Markdown
Contributor

🤔 Pourquoi ?

Affichage du spinner sur la totalité du formulaire et non seulement sur le champ qui a déclenché l'événement.

🚨 À vérifier

  • Mettre à jour le CHANGELOG_breaking_changes.md ?
  • Ajouter l'étiquette « Bug » ?

💻 Captures d'écran

Avant
image

Après
image

@celine-m-s celine-m-s self-assigned this Apr 24, 2025
@celine-m-s celine-m-s added the modifié Modifié dans le changelog. label Apr 24, 2025
@celine-m-s celine-m-s changed the title [Prolongations] WIP [Prolongations] Affichage du _spinner_ sur la totalité du formulaire Apr 24, 2025
@celine-m-s celine-m-s marked this pull request as ready for review April 24, 2025 14:52
@celine-m-s celine-m-s marked this pull request as draft April 24, 2025 14:58
If not, the has-loading-spinner class is switched on the element
holding the event (here, a form field).
Concretely, only the `reasons` field was disabled on the HTMX
request, not the entire form.
See https://htmx.org/docs/#indicators
@celine-m-s celine-m-s force-pushed the celinems/prolongation-fixes branch from 70a740b to abe8b3d Compare April 24, 2025 15:07
@celine-m-s celine-m-s marked this pull request as ready for review April 24, 2025 15:07
@celine-m-s celine-m-s added this pull request to the merge queue Apr 25, 2025
Merged via the queue into master with commit 3916a37 Apr 25, 2025
@celine-m-s celine-m-s deleted the celinems/prolongation-fixes branch April 25, 2025 07:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

modifié Modifié dans le changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants