Navigating between images within a post

[Artorias]

Thank you so much for your reply

Ok, I am sorry. I was wrong in my first post.

What I meant was :
If there are multiple pictures in one post, click to zoom does not work. it instead goes to the next image in that post.

I have tested this here https://try.discourse.org/t/image-click-to-zoom-test/961
and gives the same result. (2nd post in that thread)

[jomaxro (Joshua Rosenfeld)]

Ah, I understand now. Yes, this has been an “issue” since the lightboxing library was updated back in April. Prior to the update clicking on an image would display the image in its full resolution. However, when adding the ability to move between images in a single post, each half of the image became a click target to move forward or backwards in the sequence of images.

As noted above - this is not an easy issue to fix:

@jomaxro

Sounds great, but… this (lightboxing/gallery) isn’t a plugin, nor is it code written by Discourse. It’s an external library that Discourse uses. So either:

1. Someone needs to modify the library is imported into Discourse. This might work in the short term, but could break things if the lightbox library is updated and we try to merge it in.
2. Someone needs to commit to the lightbox library with a fix, and have it be merge in by the maintainer (who doesn’t seem very active).

[Artorias]

Ok, thank you too for your reply !
I guess i should’ve read a bit better, sorry

So in short. No solution anytime soon.

We are also having the rotating image issue. Uploaded images turn 90 degrees when posted. I am still looking into that one, but it seems to do with pictures taking by a phone and the orientation data inside the picture.
However i found a thread saying it was fixed in 1.9.x.

Btw: I am just a member on our forum

[jomaxro (Joshua Rosenfeld)]

@Artorias

rotating image issue

As far as I’m aware that one’s been fixed, but if you can reproduce the issue on try please post here in the relevant topic.

[Overgrow]

This is great feature but there is a drawback that we lose the ability to zoom-in the photo to its maximum size.

When there is only one image in the post and I click on lightboxed image it is displayed fit-to-screen. I can then click again into the image to display it in actual pixels size (browser scrolling is enabled). By clicking into image I can switch between fit-to-screen and actual-pixels.

When there are two or more images in post magnific gallery mode is turned on. I lose previously described option because when I click into the image it displays next photo.

Is there any way to disable gallery mode by some quick patch?

Would you be interested in having gallery mode optional in SiteSettings? I’d try to make a PR if there is interest…

Thanks…

[sam (Sam Saffron)]

¿Hay alguna razón para mantener esto abierto? Siento que ahora lo manejamos muy bien de forma nativa y es un poco difícil saber exactamente qué es pr-welcome aquí. (eliminando la etiqueta por ahora)

¿@Johani?

[Johani (Joe)]

Creo que este tema ha cumplido su propósito. Había tres ideas aquí.

1. Característica: Si una publicación tiene varias imágenes, muéstralas en una galería; esto se implementó hace mucho tiempo.
   FEATURE: Use image gallery functionality in magnific by gdpelican · Pull Request #4831 · discourse/discourse · GitHub

2. Error: cuando se está dentro de una galería, hacer clic en una imagen solía acercarla y navegar a la siguiente imagen. Esto se solucionó aquí
   UX: improve lightbox gallery zoom/navigation by hnb-ku · Pull Request #13500 · discourse/discourse · GitHub

3. Característica: Si navego a la última imagen de una publicación dentro de la galería, muéstrame imágenes de la siguiente publicación. No apoyo ese cambio porque sería bastante confuso.

Así que, todo bien para cerrar esto