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.

Sign up for GitHub

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

Jump to bottom

Partage des images de galerie #19

Closed
Yago opened this issue Sep 24, 2015 · 7 comments
Closed

Partage des images de galerie #19

Yago opened this issue Sep 24, 2015 · 7 comments

Comments

@Yago
Copy link
Contributor

Yago commented Sep 24, 2015

Actuellement, le partage social via Photoswipe ne conduit que vers la page la galerie. Afin, de partager une image définie, il faut :

1. Générer des pages vide pour chaque image avec les bons metas OG. Ce hack permet de partager une image définie sur Facebook/Twitter qui, une fois visitée, redirigera l'utilisateur sur la page “officielle” .

L'HTML de la page :

<html lang="fr" prefix="og: http://ogp.me/ns#" itemscope itemtype="http://schema.org/Article">
<head>
  <meta name="og:image" property="og:image" content="[IMAGE_URL]">
  <meta name="twitter:image" content="[IMAGE_URL]">
  <meta http-equiv="refresh" content="0;URL=[IMAGE_INGALLERY_URL]">
</head>
<body>
  <script>window.location.replace("[IMAGE_INGALLERY_URL]");</script>
</body>
</html>

2. Faire un trigger sur la thumbnail pour ouvrir Photoswipe s'il y a un pid comme paramètre dans l'url.

3. Redéfinir les urls de partage dans Photoswipe qui pointeront vers les page “vide”.

@lcherpit Typo3 peut générer ces pages de partage pour chaque image de galerie ?
@lcherpit
Copy link
Contributor

@Yago Si je comprend bien. il s'agirait d'une page avec un mécanisme afin d'afficher l'image partagée pour Facebook/twitter et ensuite rediriger vers la page de la galerie et déclencher le trigger qui ouvrira l'image dans photoswipe.
Oui, je pense que c'est faisable!

@Yago
Copy link
Contributor Author

Yago commented Sep 24, 2015

Un “ mécanisme” ? C'est une page utilisée uniquement pas Facebook/Twitter qui vont y lire les metas, mais quand un utilisateur s'y rend, il est redirigé vers la galerie. Du coup Photoswipe va partager les liens de ces pages-hack plutôt que ceux actuellement.

@lcherpit
Copy link
Contributor

Oui "mécanisme"... enfin c'est pour dire qu'il s'agit d'une page générée avec les bonnes metas et, je corrige: non pas pour afficher l'image.
oui, c'est faisable.

@Yago
Copy link
Contributor Author

Yago commented Sep 24, 2015

Super, moi je m'occuperai de Photoswipe du coup 😄

@Yago
Copy link
Contributor Author

Yago commented Sep 24, 2015

Je pense que pour faciliter l'implémentation des nouveaux liens dans Photoswipe, on pourrait mettre un data attribute sur l'image ou le lien de miniature, ex: <img data-sharing="http://..."

@lcherpit
Copy link
Contributor

Oui, bonne solution, un attribut data avec le lien vers la page qui contient les metas de partage de l'image et dans les metas l'image de miniatures (500px de large?) pour l'image de référence pour le partage et l'url de redirection IMAGE_INGALLERY_URL avec la partie hash pour photoswipe.
Actuellement #&gid=null&pid=4 mais peut-être qu'on pourrait ne garder que le pid car nous n'avons pas plusieurs groupe d'images.

@lcherpit
Copy link
Contributor

forké pour les modif.

lcherpit added a commit to lcherpit/styleguide that referenced this issue Dec 23, 2015
@lcherpit
VaudoiseAssurances#19 Fix: Default value should be set for gid param
fcc4c7a
lcherpit added a commit to lcherpit/styleguide that referenced this issue Dec 23, 2015
@lcherpit
VaudoiseAssurances#19 Fix lint warning
a5bc1af
lcherpit added a commit to lcherpit/styleguide that referenced this issue Dec 23, 2015
@lcherpit
VaudoiseAssurances#19 Arrange and format the code a bit
3aa616f
lcherpit added a commit to lcherpit/styleguide that referenced this issue Dec 23, 2015
@lcherpit
VaudoiseAssurances#19 Add data-sharing-url attribute on gallery items
45a760c
lcherpit added a commit to lcherpit/styleguide that referenced this issue Dec 23, 2015
@lcherpit
VaudoiseAssurances#19 Fix: Get current index from pswp gallery object
67b6184
@Yago Yago closed this as completed Jan 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@lcherpit @Yago