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)]

Há alguma razão para manter isso aberto? Sinto que agora lidamos com isso de forma excelente “out-of-the-box” e é um pouco difícil saber exatamente o que é pr-welcome aqui. (removendo a tag por enquanto)

@Johani?

[Johani (Joe)]

Acho que este tópico cumpriu seu propósito. Havia três ideias aqui.

1. Recurso: Se uma postagem tiver várias imagens, exiba-as em uma galeria - isso foi implementado há muito tempo.
   FEATURE: Use image gallery functionality in magnific by gdpelican · Pull Request #4831 · discourse/discourse · GitHub

2. Bug: quando dentro de uma galeria, clicar em uma imagem costumava ampliar e navegar para a imagem seguinte. Isso foi corrigido aqui
   UX: improve lightbox gallery zoom/navigation by hnb-ku · Pull Request #13500 · discourse/discourse · GitHub

3. Recurso: Se eu navegar para a última imagem em uma postagem dentro da galeria, mostre-me imagens da próxima postagem. Eu não apoio essa mudança porque seria bastante confuso.

Portanto, tudo bem fechar isto