maint(pat-stacks): Add updated dom to pat-update event data.

Author: thet

src/pat/stacks/stacks.js

@@ -28,7 +28,7 @@ class Pattern extends BasePattern {
     _setupStack() {
         let selected = this._currentFragment();
         const $sheets = this.$el.find(this.options.selector);
-        let $visible = [];
+        this.$active = [];
 
         if ($sheets.length < 2) {
             log.warn("Stacks pattern: must have more than one sheet.", this.$el[0]);
@@ -37,18 +37,18 @@ class Pattern extends BasePattern {
 
         if (selected) {
             try {
-                $visible = $sheets.filter("#" + selected);
+                this.$active = $sheets.filter("#" + selected);
             } catch (e) {
                 selected = undefined;
             }
         }
 
-        if (!$visible.length) {
-            $visible = $sheets.first();
-            selected = $visible[0].id;
+        if (!this.$active.length) {
+            this.$active = $sheets.first();
+            selected = this.$active[0].id;
         }
-        const $invisible = $sheets.not($visible);
-        utils.hideOrShow($visible, true, { transition: "none" }, this.name);
+        const $invisible = $sheets.not(this.$active);
+        utils.hideOrShow(this.$active, true, { transition: "none" }, this.name);
         utils.hideOrShow($invisible, false, { transition: "none" }, this.name);
         this._updateAnchors(selected);
     }
@@ -80,6 +80,8 @@ class Pattern extends BasePattern {
         this._switch(href_parts[1]);
         $(e.target).trigger("pat-update", {
             pattern: "stacks",
+            action: "attribute-changed",
+            dom: this.$active[0],
             originalEvent: e,
         });
     }
@@ -104,13 +106,13 @@ class Pattern extends BasePattern {
     }
 
     _switch(sheet_id) {
-        const $sheet = this.$el.find("#" + sheet_id);
-        if (!$sheet.length || $sheet.hasClass("visible")) {
+        this.$active = this.$el.find("#" + sheet_id);
+        if (!this.$active.length || this.$active.hasClass("visible")) {
             return;
         }
-        const $invisible = this.$el.find(this.options.selector).not($sheet);
+        const $invisible = this.$el.find(this.options.selector).not(this.$active);
         utils.hideOrShow($invisible, false, this.options, this.name);
-        utils.hideOrShow($sheet, true, this.options, this.name);
+        utils.hideOrShow(this.$active, true, this.options, this.name);
     }
 }
 

src/pat/stacks/stacks.test.js

@@ -117,18 +117,23 @@ describe("pat-stacks", function () {
             await events.await_pattern_init(pattern);
             pattern.document = { URL: document.URL };
             pattern.document.URL = "http://www.example.com";
-            const spy_trigger = jest.spyOn($.fn, "trigger");
+
+            let data = null;
+            $el.on("pat-update", (e, d) => {
+                data = d;
+            });
+
             const e = {
                 target: $el,
                 type: "click",
                 preventDefault: function () {},
                 currentTarget: { href: "http://www.example.com#s1" },
             };
             pattern._onClick(e);
-            expect(spy_trigger).toHaveBeenCalledWith(
-                "pat-update",
-                expect.objectContaining({ pattern: "stacks" })
-            );
+
+            expect(data.pattern).toBe("stacks");
+            expect(data.action).toBe("attribute-changed");
+            expect(data.dom).toBe($el[0].querySelector("#s1"));
         });
     });