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

Ajoute l'API Web Share #58

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

Conversation

QuentinWidlocher
Copy link
Contributor

Testé sur Chrome Android c'est nickel ça fonctionne très bien, les autres navigateurs ne supportant pas cette fonctionnalité n'ont aucun changement de fonctionnement.
L'url n'apparaît pas si on choisi de la cacher dans les paramètres.

Il y a un problème en revanche lors de mon test sur Firefox Nightly sur Android, les émojis ne passent pas dans le partage et on n'envoi que le titre et l'url 😕
Wordle officiel gère ce cas et n'utilise pas Web Share sur ce navigateur pour l'instant. Je prévoit un prochain commit pour gérer ce cas comme eux.

Pour tester cette PR, il faut forcément host l'appli en HTTPS (via la preview Netlify ou localtunnel par exemple)
Sinon pour juste jeter un coup d’œil au résultat : https://imgur.com/a/K8CX76n

@netlify
Copy link

netlify bot commented Apr 19, 2022

Deploy Preview for wordle-fr-louan ready!

Name Link
🔨 Latest commit 5a3d6ba
🔍 Latest deploy log https://app.netlify.com/sites/wordle-fr-louan/deploys/625e79e1f3b29d00097909c8
😎 Deploy Preview https://deploy-preview-58--wordle-fr-louan.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 settings.

@richie3366
Copy link
Collaborator

richie3366 commented Apr 19, 2022

Voici un exemple de ce que je craignais : Safari sur Mac OS ne me propose(ra) pas de copier tant qu'il ne s'agit pas d'un lien :
Capture d’écran 2022-04-19 à 16 34 35

J'ai cliqué sur "Plus", et ça m'a ouvert les Réglages avec apparemment aucun moyen d'y remédier. Peut-être existe-t-il des apps/extensions installables qui permettent d'ajouter l'action "Copier (texte)", mais je considère personnellement qu'il est peu souhaitable de considérer que les utilisateurs penseront à les chercher & les installer.
Capture d’écran 2022-04-19 à 16 37 23
Capture d’écran 2022-04-19 à 16 37 31

Je ne dis pas pour autant que c'est rédhibitoire à mes yeux. Après tout, j'utilise Chrome et j'ai cru comprendre que ce n'était pas [encore] supporté sur Mac OS. Mais je pense que si on merge cette PR, il faudra exclure les navigateurs/configurations qui ne proposent pas de fallback sur la mise en presse-papier, et les forcer à utiliser le fallback de prime abord.

J'apprécierais de connaître votre pdv sur ce point-là, à supposer qu'il ne s'accorde pas déjà avec celui que je viens d'exposer à l'instant.

Edit : même en passant par Notes, j'ai que ça qui se crée (pas de grille d'emojis)
Capture d’écran 2022-04-19 à 16 45 47

@QuentinWidlocher
Copy link
Contributor Author

Mince, c'est bien moins supporté que prévu si tous les navigateurs et OS ont leur façons de partager le contenu.

Comme dit, je pense que si on active cette fonctionnalité uniquement sur les plateformes où l'on sait que le partage fonctionne correctement (uniquement Chrome Android pour l'instant 😅) ça sera une fonctionnalité intéressante qui ne casse pas le fonctionnement actuel.

Cela dit, je ne connais pas les statistiques de Le Mot mais j'imagine que Chrome Android représente une portion non négligeable des joueurs (en tout cas moi j'en fait parti 😁)

@davidsandoz
Copy link

Sur iOS, copier dans le presse-papier est disponible avec l'API Web Share.

IMG_3F81E637D190-1

@RyanPaulGannon
Copy link

A new user left a comment. This user must be approved by a Netlify team owner before comments can be displayed.

Approve this user

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.

4 participants