Listen for transition end

[jimblue]

You're using requestAnimationFrame two times but without changing dom propriety from javascript.
I don't see how this can improve performance as you add class to animate dom element.

I can be wrong of course, if it's the case please light me candle 👍

basicLightbox/src/scripts/main.js

Line 123 in e82ed10

requestAnimationFrame(() => {

basicLightbox/src/scripts/main.js

Line 150 in e82ed10

requestAnimationFrame(() => {

[electerious]

The first requestAnimationFrame is there to ensure that the class change triggers the animation. You have to wait a frame after adding the element to the DOM. Otherwise it wouldn't do the transition to the basicLightbox--visible class.

I'm not sure if there's a reason for the second requestAnimationFrame. I would say that it's useless. Might be worth a try. We could also use a keyframe animation and avoid requestAnimationFrame completely.

[jimblue]

Ok I got it, thanks you 😄 !

Also I think you could provide a better callback here:

basicLightbox/src/scripts/main.js

Line 154 in ab2a2df

}, 410)

Why don't you listen for animation end?

[electerious]

Removed the second requestAnimationFrame: https://github.com/electerious/basicLightbox/blob/master/CHANGELOG.md#401---2018-02-23

Listening for animation end would be better. PR welcome :)