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

refactor: use TextCopyToClipboard component #661

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

Conversation

tonai
Copy link
Collaborator

@tonai tonai commented Feb 14, 2025

PR pour l'issue #635

  • modification affichage snackbar pour utiliser le composant alert
  • ajout des props label, title et children sur le composant TextCopyToClipboard (et modification de l'icône)
  • création d'un hook pour facilement pouvoir réutiliser juste la fonctionnalité de la snackbar
  • utilisation du composant sur les pages:
    • Mon compte
    • Mes clés d’accès
  • utilisation du hook sur l'onglet service d'un entrepôt

@tonai tonai force-pushed the feat/copy-to-clipboard branch from 0d13d0a to 6c4847b Compare February 14, 2025 15:34
@tonai tonai changed the title refactor: use TetCopyOoClipboard component refactor: use TextCopyToClipboard component Feb 14, 2025
@ocruze
Copy link
Member

ocruze commented Feb 17, 2025

LGTM!

Je laisse ici ces captures d'écran pour validation des textes et positionnement de l'alerte @slafayIGN

image

image

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.

2 participants