Giter Club home page Giter Club logo

Comments (21)

sachinchoolur avatar sachinchoolur commented on July 19, 2024

Hi @erbertian

I have added link for the caption.You can enable this option by setting 'captionLink=true'.
Please have a look at this link http://jsfiddle.net/sachin377/qn656/

Ragards,
Sachin N

from lightgallery.

erbertian avatar erbertian commented on July 19, 2024

Hi Sachin N,

That's great and is exactly what I was looking for thank you very much. I hope you don't mind me asking, is it also possible to include a slide count too? So it displays 01, 02, 03 etc as the slide changes, or 1 of 5, 2 of 5, 3 of 5 etc.

Many thanks again.

from lightgallery.

tammo0 avatar tammo0 commented on July 19, 2024

hi @erbertian,
i've done it but have some issues with github. because of that i created a jsfiddle with my fork for you while its not possible for me to merge:
http://jsfiddle.net/549hG/

from lightgallery.

erbertian avatar erbertian commented on July 19, 2024

Hi tammo0,

Many thanks for that. I have added the revised js and relevant style. But the count isn't correct. I have two galleries set up as 'rel1' and 'rel2' one with a single video in (the counter says 1/4 for this) the other with 8 images in (the counter says 1/16, for this one)?

Thanks again.

from lightgallery.

tammo0 avatar tammo0 commented on July 19, 2024

Hi erbertian,
can you provide a jsfiddle of your setup?
Thanks,
tammo

from lightgallery.

tammo0 avatar tammo0 commented on July 19, 2024

Hi again,
@erbertian : http://jsfiddle.net/LyDAa/
this is how it works for me.

from lightgallery.

erbertian avatar erbertian commented on July 19, 2024

Hi again tammo0

Thanks for posting the fiddle. I have managed to recreate the issue: http://jsfiddle.net/LyDAa/4/
it seems that when the options: caption:true, desc:true, captionLink:true are used it is causing the issue? would there be a workaround for this? So I could still use the above options and retain the counter?

many thanks

from lightgallery.

tammo0 avatar tammo0 commented on July 19, 2024

Hi @erbertian
i fixed your issue. Look at line 363 and add a ">" to the selector between "slider" and "div":
https://github.com/tammo0/lightGallery/blob/master/light-gallery/js/lightGallery.js#L363

from lightgallery.

erbertian avatar erbertian commented on July 19, 2024

That's great tammo0. Worked a treat! Thank you very much.

Best wishes,
erbertian

from lightgallery.

erbertian avatar erbertian commented on July 19, 2024

Hi again,

sorry I have a further issue with the links inside the lightbox. All the slides are picking up the link from the last slide. So if slide 1 has the data-link 'www.google.com', slide 2 has the data-link 'www.google.com/2', and slide 3 has the data-link 'www.google.com/3'. When the lightbox is generated all the slides will show 'www.google.com/3' as the url.

If there is a way to resolve this I would appreciate it. Many thanks for the great work again.

from lightgallery.

sachinchoolur avatar sachinchoolur commented on July 19, 2024

Hi Erbertian,

It's working fine for me.
http://jsfiddle.net/sachin377/NDm5h/1/

from lightgallery.

sachinchoolur avatar sachinchoolur commented on July 19, 2024

Hi @tammo0,

I am not able to merge your pull request. your repository seems to be unavailable.

from lightgallery.

erbertian avatar erbertian commented on July 19, 2024

Hi Sachinchoolur

Thanks, It looks like it's when you use the option: mode:"fade" it causes the problem for me. If I take that option out, the links work fine, like in your jsfiddle above.

Is there a solution to this?

from lightgallery.

sachinchoolur avatar sachinchoolur commented on July 19, 2024

HI,

We can fix this issue by setting current slide z-index to 9.
please include these css code into your document.
#lightGallery-slider.fadeM.on .current {z-index: 9;}

Tammo's pull request is still pending ('for slide count'). so i am not able to upload updated css file.

Regards,
Sachin.

from lightgallery.

tammo0 avatar tammo0 commented on July 19, 2024

@sachinchoolur, see #12 - i think it should work now. regards, tammo

from lightgallery.

sachinchoolur avatar sachinchoolur commented on July 19, 2024

@tammo0, It is working now.

from lightgallery.

erbertian avatar erbertian commented on July 19, 2024

Hi Sachin,

thanks for the current slide z-index fix. It has improved it, but the first slide still has the last slides link, but if you click to the 2nd slide and then go back to the 1st again the link is then correct.

from lightgallery.

sachinchoolur avatar sachinchoolur commented on July 19, 2024

HI Erbertian,

Yesterday i couldn't test that properly. sorry for the inconvenience.
the issue has been fixed and uploaded. please do replace lightGallery.css
here is the updated fiddle http://jsfiddle.net/sachin377/NDm5h/2/

from lightgallery.

erbertian avatar erbertian commented on July 19, 2024

Hi Sachin,

you don't need to apologise. The new fix is now working as desired. Thank you again for your excellent support and work.

Best,
erbertian

from lightgallery.

erbertian avatar erbertian commented on July 19, 2024

Thanks

from lightgallery.

tammo0 avatar tammo0 commented on July 19, 2024

hi @erbertian , please open a new issue for every problem, bug or question you have.
when someone else is looking for a solution its easier for them to find it because it relates to the topic. Also for fixing bugs or writing features its better to have different issues you can relate to.

from lightgallery.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.