-
Notifications
You must be signed in to change notification settings - Fork 4
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
Remarques diverses sur la prod #267
Comments
Pour le lien copier, ok j'ai pu reproduire sur https://tqqhd9.csb.app. Ça semble être un problème d'utilisation du navigator.clipboard sur un iframe qui n'a pas la même origine que son parent ! |
Pour le "Copier le lien" avant on utilisait le navigator.share quand celui-ci était disponible, mais comme on n'aimait pas trop, on est revenu à juste un copier dans le presse-papier |
Il faudrait demander à nos intégrateurs d'ajouter allow="clipboard-read; clipboard-write"... C'est dommage je trouve, et on peut pas partir du principe qu'ils le feront. Peut-être qu'on peut détecter les cas où ça marche pas pour repasser à un navigator share, si ça marche en iframe cross-origin, et en textArea si le share marche pas ? |
J'ajoute déja le allow="clipboard-read; clipboard-write" dans l'exemple de code pour intégrer nos iframes, ça ne coûte rien. |
-> sur quel navigateur ? Le code est correct et semble compatible avec tous les nav théoriquement.
À voir si on veut prioriser l'insertion de boutons de retour sur tous les cas comme ça ou si on part du principe que le bouton de retour du navigateur sert à ça. Perso je suis plutôt partisan du bouton retour du navigateur sauf quand les actions qui mènent à ça ressemblent à un bouton intéractif, pas à un lien hypertexte.
Corrigé !
The text was updated successfully, but these errors were encountered: