From: Geoff Appleby Date: Wed, 27 Apr 2016 22:20:52 +0000 (-0700) Subject: Toggle aria-expanded on all offcanvas triggers X-Git-Tag: v6.2.2-rc1~21^2 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=refs%2Fpull%2F8688%2Fhead;p=thirdparty%2Ffoundation%2Ffoundation-sites.git Toggle aria-expanded on all offcanvas triggers --- diff --git a/js/foundation.offcanvas.js b/js/foundation.offcanvas.js index cca11d20e..5e7b3cf8f 100644 --- a/js/foundation.offcanvas.js +++ b/js/foundation.offcanvas.js @@ -22,6 +22,7 @@ class OffCanvas { this.$element = element; this.options = $.extend({}, OffCanvas.defaults, this.$element.data(), options); this.$lastTrigger = $(); + this.$triggers = $(); this._init(); this._events(); @@ -40,7 +41,7 @@ class OffCanvas { this.$element.attr('aria-hidden', 'true'); // Find triggers that affect this element and add aria-expanded to them - $(document) + this.$triggers = $(document) .find('[data-open="'+id+'"], [data-close="'+id+'"], [data-toggle="'+id+'"]') .attr('aria-expanded', 'false') .attr('aria-controls', id); @@ -178,6 +179,8 @@ class OffCanvas { // _this._stick(); // } }); + + this.$triggers.attr('aria-expanded', 'true'); this.$element.attr('aria-hidden', 'false') .trigger('opened.zf.offcanvas'); @@ -186,7 +189,7 @@ class OffCanvas { } if (trigger) { - this.$lastTrigger = trigger.attr('aria-expanded', 'true'); + this.$lastTrigger = trigger; } if (this.options.autoFocus) { @@ -275,7 +278,7 @@ class OffCanvas { this.$exiter.removeClass('is-visible'); } - this.$lastTrigger.attr('aria-expanded', 'false'); + this.$triggers.attr('aria-expanded', 'false'); if (this.options.trapFocus) { $('[data-off-canvas-content]').removeAttr('tabindex'); }