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

Unifie le style des modales #1158

Closed
wants to merge 1 commit into from
Closed

Unifie le style des modales #1158

wants to merge 1 commit into from

Conversation

thom4parisot
Copy link
Member

@thom4parisot thom4parisot commented Dec 12, 2024

image

  • création d'article
  • partage d'article (en double dans Article.jsx ??)
  • création d'une copie d'article
  • suppression d'article
  • message d'erreur (ErrorMessageCard)
  • export d'article (session collaborative)
  • export d'article (session solo)
  • rejoindre une session collaborative
  • terminer la session collaborative
  • créer un corpus
  • supprimer un corpus
  • exporter un corpus
  • modifier un corpus
  • modifier les métadonnées d'un corpus
  • créer un espace de travail
  • quitter un espace de travail
  • gérer les membres d'un espace de travail

Deux hics :

  • pas possible de changer l'apparence d'un bouton Geist (on ne peut pas dire type="secondary comme un bouton Geist)
  • pas possible de déclencher à distance l'envoi d'un formulaire. <button form="..."> n'a pas l'air de fonctionner avec un type="button", et Geist ne permet pas de le surcharger. On n'a donc pas de moyen facile de déclencher une action en dehors du composant enfant (le contenu de la modale).

Cette librairie me laisse vraiment perplexe (cf. #1150).

refs #1149
fixes #1132

@thom4parisot thom4parisot marked this pull request as draft December 12, 2024 17:38
Copy link

netlify bot commented Dec 12, 2024

Deploy Preview for stylo-docs canceled.

Name Link
🔨 Latest commit 126d508
🔍 Latest deploy log https://app.netlify.com/sites/stylo-docs/deploys/675b1fa9002e660008faa891

Copy link

netlify bot commented Dec 12, 2024

Deploy Preview for stylo-dev ready!

Name Link
🔨 Latest commit 126d508
🔍 Latest deploy log https://app.netlify.com/sites/stylo-dev/deploys/675b1fa9101b70000841fef0
😎 Deploy Preview https://deploy-preview-1158--stylo-dev.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.

@ggrossetie
Copy link
Collaborator

Comme proposé dans #1150 je pense qu'il faut plutôt améliorer notre composant Modal avec un hook qui va bien

@thom4parisot thom4parisot deleted the fix/1132 branch December 20, 2024 09:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Homogénéiser les designs des fenêtres contextuelles
2 participants