• FreshDig

    (@zunnieforu)


    Found bug if use new type of lightbox (style 3,4) in old browser (I tested in IE9).
    Error in lightbox.js — ‘Unable to get property ‘css’ of undefined or null’, in these lines
    $inner.css({
    height: ‘calc(100% – 92px)’,
    top: ’45px’
    });
    Because $inner is null, this is my way to fix it..
    var $inner = this.$cont.find(‘.cont-inner’); /* Move this line out from if condition*/
    if (this.effectsSupport()) {
    /*var $inner = this.$cont.find(‘.cont-inner’); — This is your old line which I moved*/
    $inner.css(‘transition-timing-function’, ‘ease’);
    $inner.css(‘transition-duration’, this.settings.speed + ‘ms’);
    }

    Ps.I know less persons care about old version of browser esspecially IE, Just want to share my way for your awesome plugin. :))
    Cheers!

    • This topic was modified 8 years ago by FreshDig.
  • The topic ‘New Type of Lightbox in old browser’ is closed to new replies.