Giter Club home page Giter Club logo

Comments (6)

ooloth avatar ooloth commented on June 3, 2024 1

I'm impressed! Thanks for the quick turnaround.

The captions work great. The points you mentioned tested well in Safari, Chrome, Firefox and iOS Safari (single touch hides the caption and buttons, zooming works on mobile).

There is a slight difference in how the captions line up with the bottom of the image depending on whether the iOS Safari bottom bar is showing (see the following screenshots). Not sure if that's fixable:

img_1959
img_1961

An option to position the caption directly below the image (instead of at the bottom of the screen) would also be lovely, but it also works great without it.

Thanks again!

from halkabox.js.

ahmednooor avatar ahmednooor commented on June 3, 2024

@manouman yes you can add alt or title attributes but the script as of yet wont change it into caption in the lightbox overlay. but the alt and title attributes will be applied nonetheless according to the browser implemented specifications.

from halkabox.js.

ooloth avatar ooloth commented on June 3, 2024

Would love to see this caption feature added. Adding visible image credits to the lightbox is necessary for most of my projects.

from halkabox.js.

ahmednooor avatar ahmednooor commented on June 3, 2024

@ooloth @manouman

Thank you for your input. I have added the caption feature. Please visit the following link and test it and share your feedback. Thank you.

https://ahmednooor.github.io/halkaBox.js/testing

p.s. single touch will hide the caption and buttons. please test that too. and please see if browser based zooming works (pinching in and out) on mobile. thank you.

from halkabox.js.

ahmednooor avatar ahmednooor commented on June 3, 2024

@ooloth thanks a lot for the detailed test. the caption is position fixed to the bottom and have a semi opaque background that is why when the bottom bar of the safari appears, the screen height shortens and hence the caption moves over the image. Regarding "An option to position the caption directly below the image (instead of at the bottom of the screen)", if that is done then the height of the image will decrease relative to the height of the caption and i haven't added a custom zooming functionality that can work on desktop, yet. so this is not a viable option.
Once again thanks a lot for your input.

from halkabox.js.

ahmednooor avatar ahmednooor commented on June 3, 2024

Features added in version 1.0.0
https://github.com/ahmednooor/halkaBox.js/releases/tag/1.0.0

from halkabox.js.

Related Issues (19)

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.