luminous icon indicating copy to clipboard operation
luminous copied to clipboard

onChange doesn't work

Open andreymal opened this issue 2 years ago • 1 comments

I tried to use dist/luminous.min.js (commit https://github.com/imgix/luminous/commit/1c53f53e2471022e95ebcd69567e948b6ef225b9) and found that onChange doesn't work:

new LuminousGallery(
  document.querySelectorAll('.gallery-item a'),
  {onChange: ({ imgEl }) => alert('This alert will never be triggered')},
);

But this code works:

new LuminousGallery(
  document.querySelectorAll('.gallery-item a'),
  {M: ({ j }) => alert('It works WHAT THE HECK!!!11')},
);

The minification minified things that shouldn't have been minified, lol

andreymal avatar Jun 03 '22 11:06 andreymal

Hey @andreymal thanks for bringing this to our attention. I've made a note of this and will ask one of our engineers to investigate.

sherwinski avatar Jun 17 '22 19:06 sherwinski