Forum Replies Created

Viewing 9 replies - 1 through 9 (of 9 total)
  • I can confirm the saving settings bug has been resolved. Using WP 3.9.1.

    Many thanks for the great plugin and your quick action!

    I can’t save plugin settings either. Using a shortcode and the general plugin settings.

    Forum: Plugins
    In reply to: [MaxGalleria] No lightbox
    Thread Starter rootkowski

    (@rootkowski)

    Apologies for my late reply.

    A quick search “Find in files” in Sublimetext didn’t show any references to jquery-migrate, so I can’t say for sure that Roots removes it. On the other hand a similar search through the twentytwelve theme folder didn’t bring up any results for jquery-migrate either.

    However if I enqueue the library in my theme the lightbox effect starts working. So I’m not really sure how jquery-migrate is otherwise included.

    Forum: Plugins
    In reply to: [MaxGalleria] No lightbox
    Thread Starter rootkowski

    (@rootkowski)

    The only difference between 2012 and Roots in terms of what scripts are loaded is that 2012 loads jquery-migrate. When I added that to my Roots based theme, the lightbox effect started working as it should.

    Perhaps the plugin should list jquery-migrate as one of its dependencies, when enqueueing scripts?

    Forum: Plugins
    In reply to: [MaxGalleria] No lightbox
    Thread Starter rootkowski

    (@rootkowski)

    I don’t know why I didn’t check that earlier, anyway the lightbox effect works perfectly fine with the 2012 theme, but it breaks when I active Roots theme or my custom theme based on Roots. That’s a start ?? I’ll see what else I can find out.

    Forum: Plugins
    In reply to: [MaxGalleria] No lightbox
    Thread Starter rootkowski

    (@rootkowski)

    Hi,

    Thanks for your reply!

    Here’s a url (it’s a test site for a client). Still happening as of today, but I’m adding more galleries, not sure if that’ll trigger any changes.

    https://mortimertest.co.uk/jvm/gallery-of-results/face/facelift/

    Forum: Plugins
    In reply to: [MaxGalleria] No lightbox?

    Sorry for posting in a resolved thread.

    I have the same issue, the lightbox doesn’t kick in. I’m quite sure there are some dependencies missing because the lightbox feature works with no problems if NextGEN plugin is enabled. If I disable nextgen the chrome debugger says Uncaught TypeError: Cannot read property ‘msie’ of undefined from the fancybox pack js file referring to lines 18 and 46 of that file as anonymous function.

    There is also another error from the image-tiles.js file: Uncaught TypeError: undefined is not a function. This error refers to lines 13 and 2 of the file as anonymous function and to jquery min file. In jquery it refers to n.extend.each, n.fn.n.each, j, k.fireWith, n.extend.ready and finally to K.

    If you’d need any more info please let me know. Thanks in advance for your help!

    Thread Starter rootkowski

    (@rootkowski)

    Thanks for the info Jerry!

    I know it’s not always easy to find the time to fix bugs, so no stress ??

    I also know how useful it is to get people to test your plugins and find possible bugs. Do you host your plugin code on github or somewhere like that? Wondering if you accept code contributions? (I’m not promising anything though ??

    Cheers,
    Przemek

    It is a solution, but not the best practise hacking the plugin files. I wonder if creating your own translation file would be a better approach since the date format is specified within the __() function?

    I’m currently looking to change the date format myself to “2nd June 6.00” but I’ve no idea what date system is actually used in this plugin. Using the format characters from https://php.net/manual/en/function.date.php makes a big mess of the dates in the plugin. Any hints, please? ??

    Cheers!

    EDIT: looking through the plugin php file I found that if refers to the strftime() function ( https://www.php.net/manual/en/function.strftime.php ), so the funny codes with percentage sign are now clear ??

Viewing 9 replies - 1 through 9 (of 9 total)