diff --git a/core/modules/widgets/eventcatcher.js b/core/modules/widgets/eventcatcher.js index 1f3d96688..09f2e3ad7 100644 --- a/core/modules/widgets/eventcatcher.js +++ b/core/modules/widgets/eventcatcher.js @@ -43,66 +43,76 @@ EventWidget.prototype.render = function(parent,nextSibling) { // Assign classes this.assignDomNodeClasses(); // Add our event handler - domNode.addEventListener(this.type,function(event) { - var selector = self.getAttribute("selector"), - actions = self.getAttribute("actions"), - selectedNode = event.target, - selectedNodeRect, - catcherNodeRect, - variables = {}; - if(selector) { - // Search ancestors for a node that matches the selector - while(!selectedNode.matches(selector) && selectedNode !== domNode) { - selectedNode = selectedNode.parentNode; - } - // If we found one, copy the attributes as variables, otherwise exit - if(selectedNode.matches(selector)) { - $tw.utils.each(selectedNode.attributes,function(attribute) { - variables["dom-" + attribute.name] = attribute.value.toString(); - }); - //Add a variable with a popup coordinate string for the selected node - variables["tv-popup-coords"] = "(" + selectedNode.offsetLeft + "," + selectedNode.offsetTop +"," + selectedNode.offsetWidth + "," + selectedNode.offsetHeight + ")"; - - //Add variables for offset of selected node - variables["tv-selectednode-posx"] = selectedNode.offsetLeft.toString(); - variables["tv-selectednode-posy"] = selectedNode.offsetTop.toString(); - variables["tv-selectednode-width"] = selectedNode.offsetWidth.toString(); - variables["tv-selectednode-height"] = selectedNode.offsetHeight.toString(); + $tw.utils.each(this.types,function(type) { + domNode.addEventListener(type,function(event) { + var selector = self.getAttribute("selector"), + actions = self.getAttribute("actions-"+type), + selectedNode = event.target, + selectedNodeRect, + catcherNodeRect, + variables = {}; + if(selector) { + // Search ancestors for a node that matches the selector + while(!selectedNode.matches(selector) && selectedNode !== domNode) { + selectedNode = selectedNode.parentNode; + } + // If we found one, copy the attributes as variables, otherwise exit + if(selectedNode.matches(selector)) { + $tw.utils.each(selectedNode.attributes,function(attribute) { + variables["dom-" + attribute.name] = attribute.value.toString(); + }); + //Add a variable with a popup coordinate string for the selected node + variables["tv-popup-coords"] = "(" + selectedNode.offsetLeft + "," + selectedNode.offsetTop +"," + selectedNode.offsetWidth + "," + selectedNode.offsetHeight + ")"; + + //Add variables for offset of selected node + variables["tv-selectednode-posx"] = selectedNode.offsetLeft.toString(); + variables["tv-selectednode-posy"] = selectedNode.offsetTop.toString(); + variables["tv-selectednode-width"] = selectedNode.offsetWidth.toString(); + variables["tv-selectednode-height"] = selectedNode.offsetHeight.toString(); - //Add variables for event X and Y position relative to selected node - selectedNodeRect = selectedNode.getBoundingClientRect(); - variables["event-fromselected-posx"] = (event.clientX - selectedNodeRect.left).toString(); - variables["event-fromselected-posy"] = (event.clientY - selectedNodeRect.top).toString(); + //Add variables for event X and Y position relative to selected node + selectedNodeRect = selectedNode.getBoundingClientRect(); + variables["event-fromselected-posx"] = (event.clientX - selectedNodeRect.left).toString(); + variables["event-fromselected-posy"] = (event.clientY - selectedNodeRect.top).toString(); - //Add variables for event X and Y position relative to event catcher node - catcherNodeRect = self.domNode.getBoundingClientRect(); - variables["event-fromcatcher-posx"] = (event.clientX - catcherNodeRect.left).toString(); - variables["event-fromcatcher-posy"] = (event.clientY - catcherNodeRect.top).toString(); - } else { - return false; - } - } - // Execute our actions with the variables - if(actions) { - // Add a variable for the modifier key - variables.modifier = $tw.keyboardManager.getEventModifierKeyDescriptor(event); - // Add a variable for the mouse button - if("button" in event) { - if(event.button === 0) { - variables["event-mousebutton"] = "left"; - } else if(event.button === 1) { - variables["event-mousebutton"] = "middle"; - } else if(event.button === 2) { - variables["event-mousebutton"] = "right"; + //Add variables for event X and Y position relative to event catcher node + catcherNodeRect = self.domNode.getBoundingClientRect(); + variables["event-fromcatcher-posx"] = (event.clientX - catcherNodeRect.left).toString(); + variables["event-fromcatcher-posy"] = (event.clientY - catcherNodeRect.top).toString(); + } else { + return false; } } - self.invokeActionString(actions,self,event,variables); - event.preventDefault(); - event.stopPropagation(); - return true; - } - return false; - },false); + // Execute our actions with the variables + if(actions) { + // Add a variable for the modifier key + variables.modifier = $tw.keyboardManager.getEventModifierKeyDescriptor(event); + // Add a variable for the mouse button + if("button" in event) { + if(event.button === 0) { + variables["event-mousebutton"] = "left"; + } else if(event.button === 1) { + variables["event-mousebutton"] = "middle"; + } else if(event.button === 2) { + variables["event-mousebutton"] = "right"; + } + } + variables["event-type"] = event.type; + if(typeof event.detail === "object" && !!event.detail) { + $tw.utils.each(event.detail,function(detailValue,detail) { + variables["event-detail-" + detail] = detailValue; + }); + } else if(!!event.detail) { + variables["event-detail"] = event.detail; + } + self.invokeActionString(actions,self,event,variables); + event.preventDefault(); + event.stopPropagation(); + return true; + } + return false; + },false); + }); // Insert element parent.insertBefore(domNode,nextSibling); this.renderChildren(domNode,null); @@ -115,7 +125,7 @@ Compute the internal state of the widget EventWidget.prototype.execute = function() { var self = this; // Get attributes that require a refresh on change - this.type = this.getAttribute("type"); + this.types = this.getAttribute("events","").split(" "); this.elementTag = this.getAttribute("tag"); // Make child widgets this.makeChildWidgets(); @@ -132,7 +142,7 @@ Selectively refreshes the widget if needed. Returns true if the widget or any of */ EventWidget.prototype.refresh = function(changedTiddlers) { var changedAttributes = this.computeAttributes(); - if(changedAttributes.type || changedAttributes["tag"]) { + if(changedAttributes["events"] || changedAttributes["tag"]) { this.refreshSelf(); return true; } else if(changedAttributes["class"]) { diff --git a/editions/tw5.com/tiddlers/widgets/EventCatcherWidget.tid b/editions/tw5.com/tiddlers/widgets/EventCatcherWidget.tid index 28d2c9e9a..9a4b498c5 100644 --- a/editions/tw5.com/tiddlers/widgets/EventCatcherWidget.tid +++ b/editions/tw5.com/tiddlers/widgets/EventCatcherWidget.tid @@ -1,5 +1,5 @@ created: 20201123113532200 -modified: 20201123172753436 +modified: 20201202200719126 tags: Widgets title: EventCatcherWidget type: text/vnd.tiddlywiki @@ -11,10 +11,10 @@ The event catcher widget traps JavaScript events dispatched within its child con In order for the events to be trapped they must: -* be of the type specified as a parameter to the event catcher widget. +* be of one of the types specified as a parameter to the event catcher widget. * arise within a DOM node matching the selector specified as a parameter to the widget. -Use of the event catcher widget is useful when using large numbers of other trigger widgets such as the ButtonWidget is causing performance problems. The workflow it enables is akin to what is referred to as "event delegation" in JavaScript parlance. +Use of the event catcher widget is beneficial when using large numbers of other trigger widgets such as the ButtonWidget is causing performance problems. The workflow it enables is akin to what is referred to as "event delegation" in JavaScript parlance. //This is an advanced widget intended to be used by those familiar with HTML, CSS and JavaScript.// @@ -23,9 +23,9 @@ Use of the event catcher widget is useful when using large numbers of other trig The content of the `<$eventcatcher>` widget is displayed normally. |!Attribute |!Description | -|type |The JavaScript event type to be trapped, for example "click", or "dblclick" | +|events |Space separated list of JavaScript events to be trapped, for example "click" or "click dblclick" | |selector |A CSS selector. Only events originating inside a DOM node with this selector will be trapped. | -|actions |Action strings to be invoked when a matching event is trapped | +|actions-* |Action strings to be invoked when a matching event is trapped. Each event is mapped to an action attribute name of the form `actions-"event"` where `event` represents the type of the event. For example: `actions-click` or `actions-dblclick` | |class |An optional CSS class name to be assigned to the HTML element | |tag |Optional. The html element the widget creates to capture the events, defaults to:
» `span` when parsed in inline-mode
» `div` when parsed in block-mode | @@ -37,6 +37,8 @@ The following variables are made available to the actions: |`dom-*` |All DOM attributes of the node matching the given selector are made available as variables, with the prefix `dom-` | |`modifier` |The [[modifier Variable]] contains the Modifier Key held during the event (can be "normal", "ctrl", "shift", "alt" or combinations thereof) | |`event-mousebutton`|The mouse button (if any) used to trigger the event (can be "left", "right" or "middle"). Note that not all event types support the mousebutton property | +|`event-type`|The type property of the JavaScript event | +|`event-detail-*`|Any properties in the detail attribute of the event are made available with the prefix `event-detail-` | |`tv-popup-coords`|A co-ordinate string that can be used with the ActionPopupWidget to trigger a popup at the DOM node matching the selector where the event originated | |`tv-selectednode-posx`|`x` offset position of the selected DOM node | |`tv-selectednode-posy`|`y` offset position of the selected DOM node | @@ -52,11 +54,15 @@ The following variables are made available to the actions: This example uses the ActionLogWidget and will log the `data-item-id` attribute of the clicked DOM node to the browser's JavaScript [[console|Web Developer Tools]] ``` -\define myactions() -<$action-log item=<>/> +\define clickactions() +<$action-log item=<> event=<>/> \end -<$eventcatcher type="click" selector=".item" actions=<> tag="div"> +\define contextmenu-actions() +<$action-log item=<> event=<>/> +\end + +<$eventcatcher events="click contextmenu" selector=".item" actions-click=<> actions-contextmenu=<> tag="div">
Click events here will be trapped @@ -74,6 +80,6 @@ Not here And here
-""" + ```