From c95a6cf4094b3e7c2db774da9613f0db3aae5315 Mon Sep 17 00:00:00 2001 From: Chris Sherman Date: Thu, 30 Apr 2015 16:35:09 -0400 Subject: [PATCH] Update foundation.reveal.js Ensure open event has a target before accessing the 'data-reveal-id' attribute. I ran into target === undefined after calling $('#myModal').foundation('reveal', 'open'); --- js/foundation/foundation.reveal.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/js/foundation/foundation.reveal.js b/js/foundation/foundation.reveal.js index 82321464e..fca6d5b21 100644 --- a/js/foundation/foundation.reveal.js +++ b/js/foundation/foundation.reveal.js @@ -155,7 +155,7 @@ settings = settings || this.settings; - if (modal.hasClass('open') && target.attr('data-reveal-id') == modal.attr('id')) { + if (modal.hasClass('open') && target !== undefined && target.attr('data-reveal-id') == modal.attr('id')) { return self.close(modal); } -- 2.47.2