From: Daniel Ruf Date: Sat, 16 Jun 2018 07:18:46 +0000 (+0200) Subject: Use pull request #10944 from DanielRuf/fix/offcanvas-trigger-opened-later for v6.5.0 X-Git-Tag: v6.5.0-rc.1^2~128 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=11d03de4c9645544c8d2938d429033292fbea457;p=thirdparty%2Ffoundation%2Ffoundation-sites.git Use pull request #10944 from DanielRuf/fix/offcanvas-trigger-opened-later for v6.5.0 f67350326 fix: trigger opened.zf.offcanvas at the end to prevent race conditions Signed-off-by: Nicolas Coden --- diff --git a/js/foundation.offcanvas.js b/js/foundation.offcanvas.js index 995bf91d4..080a18536 100644 --- a/js/foundation.offcanvas.js +++ b/js/foundation.offcanvas.js @@ -280,6 +280,7 @@ class OffCanvas extends Plugin { * @param {Object} event - Event object passed from listener. * @param {jQuery} trigger - element that triggered the off-canvas to open. * @fires OffCanvas#opened + * @todo also trigger 'open' event? */ open(event, trigger) { if (this.$element.hasClass('is-open') || this.isRevealed) { return; } @@ -301,15 +302,10 @@ class OffCanvas extends Plugin { this.$element.siblings('[data-off-canvas-content]').css('transition-duration', ''); } - /** - * Fires when the off-canvas menu opens. - * @event OffCanvas#opened - */ this.$element.addClass('is-open').removeClass('is-closed'); this.$triggers.attr('aria-expanded', 'true'); - this.$element.attr('aria-hidden', 'false') - .trigger('opened.zf.offcanvas'); + this.$element.attr('aria-hidden', 'false'); this.$content.addClass('is-open-' + this.position); @@ -348,6 +344,12 @@ class OffCanvas extends Plugin { } this._addContentClasses(); + + /** + * Fires when the off-canvas menu opens. + * @event OffCanvas#opened + */ + this.$element.trigger('opened.zf.offcanvas'); } /**