LouanBen / wordle-fr

"Le Mot" un Wordle en français.
https://wordle.louan.me
GNU General Public License v3.0
127 stars 40 forks source link

Ajoute l'API Web Share #58

Open QuentinWidlocher opened 2 years ago

QuentinWidlocher commented 2 years ago

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 :confused:
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[bot] commented 2 years ago

Deploy Preview for wordle-fr-louan ready!

Name Link
Latest commit 5a3d6baa4889572031cc7ffedc208c0f84e7918c
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 commented 2 years ago

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 commented 2 years ago

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 :sweat_smile:) ç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 :grin:)

davidsandoz commented 2 years ago

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

IMG_3F81E637D190-1

RyanPaulGannon commented 2 years ago

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

Approve this user