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

UX/UI: Mise a jour du thème vers la v2.5.9 #5452

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

hellodeloo
Copy link
Contributor

@hellodeloo hellodeloo commented Jan 23, 2025

🤔 Pourquoi ?

Mise a jour du thème pour :

  • corriger un problème z-index des .tooltip dans les .dropdown
  • modifier le fonctionnement de la classe utilitaire .has-spinner-loading afin qu'elle se comporte différemment selon qu'elle est appliquée sur un <div> ou sur un .btn. (Sur une idée de @leo-naeka ) Plus de précisions dans le ZeroHeight
  • suppression des règles et de la classe .has-btn-with-spinner-loading-text, maintenant inutile car gérée dans le thème

@hellodeloo hellodeloo self-assigned this Jan 23, 2025
@hellodeloo hellodeloo changed the title feat: update theme to v2.5.9 and remove spinner-loading class now us… UX/UI: Mise a jour du thème vers la v2.5.9 Jan 23, 2025
@hellodeloo hellodeloo added the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Jan 23, 2025
Copy link

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

@hellodeloo
Copy link
Contributor Author

@leo-naeka Pourrais-tu checker que tes boutons rdv-i fonctionnent encore. Je ne retrouve pas comment les afficher. Thx

@leo-naeka
Copy link
Contributor

@hellodeloo c'est super d'avoir intégré au thème !

Je ne constate qu'un petit souci sur la taille du bouton lors du state loading :

Enregistrement.de.l.ecran.2025-01-24.a.14.55.39.mov
Enregistrement.de.l.ecran.2025-01-24.a.14.56.16.mov

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é
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants