Skip to content

UX/UI: Correction UI du spinner sur les switch#5552

Merged
hellodeloo merged 1 commit into
masterfrom
deloo/fix-has-spinner-on-switch
Feb 6, 2025
Merged

UX/UI: Correction UI du spinner sur les switch#5552
hellodeloo merged 1 commit into
masterfrom
deloo/fix-has-spinner-on-switch

Conversation

@hellodeloo

@hellodeloo hellodeloo commented Feb 6, 2025

Copy link
Copy Markdown
Contributor

🤔 Pourquoi ?

La classe utilitaire appliquée pendant les requêtes htmx (.has-spinner-loading) sur un .form-swith (ou sur un parent contenant un .form-swith) méritait une amélioration visuelle

🍰 Comment ?

Ajout de règles css spécifiques à ce cas

Ou ça ?

ici -> https://c1-review-deloo-fix-has-spinner-on-switch.cleverapps.io/company/job_description_list

💻 Captures d'écran

Avant
capture 2025-02-06 à 10 09 10

Apres
capture 2025-02-06 à 10 09 36

@hellodeloo hellodeloo self-assigned this Feb 6, 2025
@hellodeloo hellodeloo changed the title feat: update theme to v2.6.6 UX/UI: Correction UI du spinner sur les switch Feb 6, 2025
@hellodeloo hellodeloo added the modifié Modifié dans le changelog. label Feb 6, 2025
@notion-workspace

Copy link
Copy Markdown

@hellodeloo hellodeloo added the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Feb 6, 2025
@github-actions

github-actions Bot commented Feb 6, 2025

Copy link
Copy Markdown

🥁 La recette jetable est prête ! 👉 Je veux tester cette PR !

@hellodeloo
hellodeloo force-pushed the deloo/fix-has-spinner-on-switch branch from a566794 to 0ff6fd2 Compare February 6, 2025 09:35
@hellodeloo
hellodeloo added this pull request to the merge queue Feb 6, 2025
Merged via the queue into master with commit 6d5a0b8 Feb 6, 2025
@hellodeloo
hellodeloo deleted the deloo/fix-has-spinner-on-switch branch February 6, 2025 10:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC modifié Modifié dans le changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants