@oracle/oraclejet 10.1.7 → 10.1.8

package/dist/js/libs/oj/debug/ojjquery-hammer.js

@@ -34,6 +34,7 @@ define(['ojs/ojlogger', 'hammerjs', 'jquery'], function (Logger, Hammer, $) { 'u
               hammer.destroy();
               $el.removeData('ojHammer');
             }
+            $el.off('remove.ojHammer');
           });
 
         default:
@@ -41,7 +42,13 @@ define(['ojs/ojlogger', 'hammerjs', 'jquery'], function (Logger, Hammer, $) { 'u
           return this.each(function () {
             var $el = $(this);
             if (!$el.data('ojHammer')) {
-              $el.data('ojHammer', new Hammer.Manager($el[0], options));
+              const mgr = new Hammer.Manager($el[0], options);
+              $el.on('remove.ojHammer', evt => {
+                if (evt.target === $el[0]) {
+                  mgr.destroy();
+                }
+              });
+              $el.data('ojHammer', mgr);
             }
           });
       }

package/dist/js/libs/oj/debug_es5/ojcomponentcore.js

@@ -236,6 +236,36 @@ define(['exports', 'jqueryui-amd/widget', 'jqueryui-amd/unique-id', 'jqueryui-am
       });
     }
   };
+  /**
+   * Releases resources that would otherwise leak memory if they were not released
+   * when the component's dom is removed.
+   * For example, Hammer events are put on the document and will result in
+   * detached dom memory leak if not removed.
+   * @private
+   */
+
+
+  ComponentMessaging.prototype.releaseResources = function () {
+    if (this._activated) {
+      $.each(this._strategies, function (i, strategy) {
+        strategy.releaseResources();
+      });
+    }
+  };
+  /**
+   * Symmetrical method to releaseResources. Sets up resources that get
+   * removed in releaseResources.
+   * @private
+   */
+
+
+  ComponentMessaging.prototype.setupResources = function () {
+    if (this._activated) {
+      $.each(this._strategies, function (i, strategy) {
+        strategy.setupResources();
+      });
+    }
+  };
   /**
    * Creates a messaging strategy for the specified type, initializing it with the options provided.
    * @param {string|number} type defined by oj.ComponentMessaging._STRATEGY_TYPE. For example,
@@ -767,6 +797,20 @@ define(['exports', 'jqueryui-amd/widget', 'jqueryui-amd/unique-id', 'jqueryui-am
 
 
   MessagingStrategy.prototype.close = function () {};
+  /**
+   *
+   * @private
+   */
+
+
+  MessagingStrategy.prototype.setupResources = function () {};
+  /**
+   *
+   * @private
+   */
+
+
+  MessagingStrategy.prototype.releaseResources = function () {};
   /**
    * Reinitializes with the new display options and updates component messaging using the new content.
    *
@@ -2536,6 +2580,23 @@ define(['exports', 'jqueryui-amd/widget', 'jqueryui-amd/unique-id', 'jqueryui-am
 
   oj._registerLegacyNamespaceProp('CustomElementBridge', CustomElementBridge);
 
+  var getFocusEventPropagator = function getFocusEventPropagator(element, type) {
+    return function (event) {
+      // Ensure that the target is the custom element, not the inner element, so create
+      // a new event and dispatch on the custom element.
+      element.dispatchEvent(new FocusEvent(type, {
+        relatedTarget: event.relatedTarget
+      }));
+    };
+  };
+
+  var getTeardownFunction = function getTeardownFunction(element, focusPropagator, blurPropagator) {
+    return function () {
+      element.removeEventListener('focus', focusPropagator);
+      element.removeEventListener('blur', blurPropagator);
+    };
+  };
+
   oj.CollectionUtils.copyInto(CustomElementBridge.proto, {
     // Provides a promise for JET's Knockout throttling timeout
     // when knockout is used for the component
@@ -2700,22 +2761,7 @@ define(['exports', 'jqueryui-amd/widget', 'jqueryui-amd/unique-id', 'jqueryui-am
       } // Setup blur/focus listeners on inner element so we can trigger on the root custom element for 
 
 
-      var getFocusEventPropagator = function getFocusEventPropagator(type) {
-        return function (event) {
-          // Ensure that the target is the custom element, not the inner element, so create
-          // a new event and dispatch on the custom element.
-          element.dispatchEvent(new FocusEvent(type, {
-            relatedTarget: event.relatedTarget
-          }));
-        };
-      };
-
-      var focusElem = this._WIDGET_INSTANCE.__getFocusElement();
-
-      if (focusElem && focusElem !== element) {
-        focusElem.addEventListener('focus', getFocusEventPropagator('focus'));
-        focusElem.addEventListener('blur', getFocusEventPropagator('blur'));
-      }
+      this._setupFocusPropagation(element);
     },
     DefineMethodCallback: function DefineMethodCallback(proto, method, methodMeta) {
       // eslint-disable-next-line no-param-reassign
@@ -2833,11 +2879,15 @@ define(['exports', 'jqueryui-amd/widget', 'jqueryui-amd/unique-id', 'jqueryui-am
       if (Components.__GetWidgetConstructor(this._WIDGET_ELEM) && this._WIDGET_INSTANCE) {
         this._WIDGET_INSTANCE.__handleDisconnected();
       }
+
+      this._teardownFocusPropagation(element);
     },
     HandleReattached: function HandleReattached(element) {
       // Invoke callback on the superclass
       oj.BaseCustomElementBridge.proto.HandleReattached.call(this, element);
 
+      this._setupFocusPropagation(element);
+
       if (this._WIDGET_INSTANCE) {
         this._WIDGET_INSTANCE.__handleConnected();
       }
@@ -3038,6 +3088,25 @@ define(['exports', 'jqueryui-amd/widget', 'jqueryui-amd/unique-id', 'jqueryui-am
       }
 
       return isEvent;
+    },
+    _setupFocusPropagation: function _setupFocusPropagation(element) {
+      // Setup blur/focus listeners on inner element so we can trigger on the root custom element for 
+      var focusElem = this._WIDGET_INSTANCE.__getFocusElement();
+
+      if (focusElem && focusElem !== element) {
+        var focusPropagator = getFocusEventPropagator(element, 'focus');
+        var blurPropagator = getFocusEventPropagator(element, 'blur');
+        focusElem.addEventListener('focus', focusPropagator);
+        focusElem.addEventListener('blur', blurPropagator);
+        this._teardownFocus = getTeardownFunction(focusElem, focusPropagator, blurPropagator);
+      }
+    },
+    _teardownFocusPropagation: function _teardownFocusPropagation() {
+      if (this._teardownFocus) {
+        this._teardownFocus();
+
+        this._teardownFocus = null;
+      }
     }
   });
   /** ***********************/

package/dist/js/libs/oj/debug_es5/ojcore-base.js

@@ -59,13 +59,13 @@ function _typeof(obj) { "@babel/helpers - typeof"; return _typeof = "function" =
      * @global
      * @member {string} version JET version numberr
      */
-    version: '10.1.7',
+    version: '10.1.8',
 
     /**
      * @global
      * @member {string} revision JET source code revision number
      */
-    revision: '2022-02-17_13-46-45',
+    revision: '2022-05-03_19-21-11',
     // This function is only meant to be used outside the library, so quoting the name
     // to avoid renaming is appropriate
     noConflict: function noConflict() {

package/dist/js/libs/oj/debug_es5/ojdatetimepicker.js

@@ -2622,8 +2622,15 @@ define(['ojs/ojcore', 'jquery', 'ojs/ojeditablevalue', 'ojs/ojinputtext', 'ojs/o
       childDiv.setAttribute('aria-describedby', this._GetSubId(this._DATEPICKER_DESCRIPTION_ID));
       childDiv.className = 'oj-datepicker-content';
       dpDiv.appendChild(childDiv);
-      this._dpDiv = bindHover($(dpDiv));
-      document.body.appendChild(dpDiv); // @HTMLUpdateOK
+      this._dpDiv = bindHover($(dpDiv)); // Append the dpDiv to the CE wrapper and for widgets simply to the body
+
+      if (this._IsCustomElement()) {
+        this._dpDiv.appendTo(this.OuterWrapper); // @HTMLUpdateOK
+
+      } else {
+        this._dpDiv.appendTo(document.body); // @HTMLUpdateOK
+
+      }
     },
     _createPopupDpDiv: function _createPopupDpDiv() {
       var self = this;
@@ -2729,15 +2736,11 @@ define(['ojs/ojcore', 'jquery', 'ojs/ojeditablevalue', 'ojs/ojinputtext', 'ojs/o
         // http://bugs.jqueryui.com/ticket/7552 - A Datepicker created on a detached div has zero height
 
         this._dpDiv.css('display', 'block');
-
-        this._registerSwipeHandler();
       } else {
         this._processReadOnlyKeyboardEdit();
 
         if (this.options.readOnly !== true) {
           this._attachTrigger();
-
-          this._registerSwipeHandler();
         }
       } // attach active state change handlers
 
@@ -3021,6 +3024,10 @@ define(['ojs/ojcore', 'jquery', 'ojs/ojeditablevalue', 'ojs/ojinputtext', 'ojs/o
         }
       }
 
+      if (this._dpDiv && (this._isInLine || this.options.readOnly !== true)) {
+        this._registerSwipeHandler();
+      }
+
       return this._super();
     },
 
@@ -3070,6 +3077,8 @@ define(['ojs/ojcore', 'jquery', 'ojs/ojeditablevalue', 'ojs/ojinputtext', 'ojs/o
       // for non custom element, this is done as first step in destroy
       if (this._IsCustomElement()) {
         this._cleanUpListeners();
+
+        this._unregisterSwipeHandler();
       }
 
       return this._super();
@@ -4347,6 +4356,11 @@ define(['ojs/ojcore', 'jquery', 'ojs/ojeditablevalue', 'ojs/ojinputtext', 'ojs/o
         });
       }
     },
+    _unregisterSwipeHandler: function _unregisterSwipeHandler() {
+      if (this._dpDiv) {
+        this._dpDiv.off('swiperight swipeleft').ojHammer('destroy');
+      }
+    },
 
     /**
      * Generate the HTML for the current state of the date picker.
@@ -8779,7 +8793,8 @@ define(['ojs/ojcore', 'jquery', 'ojs/ojeditablevalue', 'ojs/ojinputtext', 'ojs/o
       }
     },
     _getPrependNode: function _getPrependNode() {
-      return this._isIndependentInput() ? $('body') : $('.oj-popup-content', this._datePickerComp.widget._popUpDpDiv.ojPopup('widget'));
+      var independentPrependNode = this._IsCustomElement() ? $(this.OuterWrapper) : $('body');
+      return this._isIndependentInput() ? independentPrependNode : $('.oj-popup-content', this._datePickerComp.widget._popUpDpDiv.ojPopup('widget'));
     },
 
     /**
@@ -9219,7 +9234,9 @@ define(['ojs/ojcore', 'jquery', 'ojs/ojeditablevalue', 'ojs/ojinputtext', 'ojs/o
         var children = group.children;
 
         for (var index = 0; index < children.length; index++) {
-          wheel = children[index];
+          wheel = children[index]; // This calls Wheel#destroy() and this calls off on each
+          // event listener and destroys Hammer.
+
           wheel.ojDestroy();
         }
       }
@@ -9255,10 +9272,15 @@ define(['ojs/ojcore', 'jquery', 'ojs/ojeditablevalue', 'ojs/ojinputtext', 'ojs/o
      * @override
      */
     _ReleaseResources: function _ReleaseResources() {
-      // for non custom element, this is done as first step in destroy
-      if (this._IsCustomElement()) {
-        this._cleanUpListeners();
-      }
+      // Since destroy calls _ReleaseResources, we do here for both CustomElement and non CustomElement
+      // instead of doing it here for CustomElement and in destroy which is only for non CustomElements.
+      this._removeResizePopupBind(); // Note: We do not have a symmetrical 'createHammer' call in SetupResources.
+      // We create the timepicker wheels that have the Hammer events
+      // when the user clicks on the timepicker icon, or when
+      // the component's options change.
+
+
+      this._destroyHammer();
 
       return this._super();
     },
@@ -9286,10 +9308,6 @@ define(['ojs/ojcore', 'jquery', 'ojs/ojeditablevalue', 'ojs/ojinputtext', 'ojs/o
         this._triggerNode.remove();
       }
 
-      this._cleanUpListeners();
-
-      this._destroyHammer();
-
       if (this._wheelPicker) {
         this._wheelPicker.remove();
 
@@ -9305,7 +9323,7 @@ define(['ojs/ojcore', 'jquery', 'ojs/ojeditablevalue', 'ojs/ojinputtext', 'ojs/o
      * @ignore
      * @private
      */
-    _cleanUpListeners: function _cleanUpListeners() {
+    _removeResizePopupBind: function _removeResizePopupBind() {
       if (this._resizePopupBind) {
         window.removeEventListener('resize', this._resizePopupBind);
       }
@@ -12063,6 +12081,60 @@ define(['ojs/ojcore', 'jquery', 'ojs/ojeditablevalue', 'ojs/ojinputtext', 'ojs/o
       return retVal;
     },
 
+    /**
+     * @ignore
+     * @protected
+     * @override
+     * @memberof oj.ojInputDateTime
+     */
+    _ReleaseResources: function _ReleaseResources() {
+      if (this._IsCustomElement()) {
+        // Trigger the ojInputTime's _ReleaseResources() using the ojInputTime instance.
+        // this._timePicker.ojInputTime('destroy') will not work.
+        // jqueryui-base#destroy will throw an error because IsCustomElement() is true
+        // and you cannot call destroy when IsCustomElement is true.
+        if (this._timePicker) {
+          var timePickerInstance = this._timePicker.ojInputTime('instance');
+
+          if (timePickerInstance) {
+            timePickerInstance._ReleaseResources();
+
+            this._timePickerResourcesReleased = true;
+          }
+        }
+      }
+
+      return this._super();
+    },
+
+    /**
+     * @protected
+     * @override
+     * @instance
+     * @memberof! oj.ojInputDateTime
+     */
+    _SetupResources: function _SetupResources() {
+      var ret = this._superApply(arguments);
+
+      if (this._IsCustomElement()) {
+        // oj-input-date-time's AfterCreate calls createTimepicker, which creates ojInputTime() which
+        // calls InputTime's _SetupResources.
+        // Then oj-input-date-time's SetupResources is called. Only call ojInputTime's _SetupResources
+        // if they were released in ReleaseResources, so check the flag.
+        if (this._timePickerResourcesReleased && this._timePicker) {
+          var timePickerInstance = this._timePicker.ojInputTime('instance');
+
+          if (timePickerInstance) {
+            timePickerInstance._SetupResources();
+
+            this._timePickerResourcesReleased = false;
+          }
+        }
+      }
+
+      return ret;
+    },
+
     /**
      * @ignore
      * @protected
@@ -12076,6 +12148,8 @@ define(['ojs/ojcore', 'jquery', 'ojs/ojeditablevalue', 'ojs/ojinputtext', 'ojs/o
     },
 
     /**
+     * Called from _destroy. This is for widgets. Cleaning up custom
+     * elements is done in ReleaseResources
      * @ignore
      * @private
      * @memberof oj.ojInputDateTime

package/dist/js/libs/oj/debug_es5/ojdvt-toolkit.js

@@ -11836,27 +11836,13 @@ define(['exports', 'ojs/ojthemeutils'], function (exports, ThemeUtils) {
 
         width = TextUtils._getCanvasTextWidth(context, textString, cssStyle);
       } else {
-        if (Agent.browser === 'chrome') {
-          if (TextUtils._letterSpacing != letterSpacing || TextUtils._wordSpacing != wordSpacing) {
-            TextUtils._clearCanvasContext();
+        var text = new TextUtils._rep_text_constructor(context, textString);
+        text.setCSSStyle(cssStyle); // Add to the stage to obtain correct measurements
 
-            TextUtils._setSpacing(wordSpacing, letterSpacing);
-          }
-
-          TextUtils._storeCanvasContext();
-
-          TextUtils._attachCanvas(context);
-
-          width = TextUtils._getCanvasTextWidth(context, textString, cssStyle);
-        } else {
-          var text = new TextUtils._rep_text_constructor(context, textString);
-          text.setCSSStyle(cssStyle); // Add to the stage to obtain correct measurements
-
-          var stage = text.getCtx().getStage();
-          stage.addChild(text);
-          width = text.GetSvgDimensions().w;
-          stage.removeChild(text);
-        }
+        var stage = text.getCtx().getStage();
+        stage.addChild(text);
+        width = text.GetSvgDimensions().w;
+        stage.removeChild(text);
       }
 
       TextUtils._cachedTextWidth[cacheKey] = width;

package/dist/js/libs/oj/debug_es5/ojeditablevalue.js

@@ -3119,6 +3119,49 @@ define(['exports', 'ojs/ojcore-base', 'ojs/ojcomponentcore', 'jquery', 'ojs/ojla
       }
     },
 
+    /**
+     * Override to release resources held by this component.
+     * It is important to unregister Hammer events
+     * since they live outside of the component's DOM and would cause Detached Dom nodes
+     * when the component is removed.
+     * @memberof oj.editableValue
+     * @instance
+     * @protected
+     */
+    _ReleaseResources: function _ReleaseResources() {
+      this._super();
+
+      if (this._IsCustomElement()) {
+        // componentMessaging#releaseResources removes any resources that would
+        // otherwise cause a memory leak, like Hammer.
+        // We do not deactivate component messaging because that is a bigger change, and
+        // more risky to backport.
+        this._getComponentMessaging().releaseResources();
+
+        this._hammerIsUnregistered = true;
+      }
+    },
+
+    /**
+     * @memberof oj.editableValue
+     * @instance
+     * @protected
+     */
+    _SetupResources: function _SetupResources() {
+      this._super(); // Since initComponentMessaging gets called during component creation, and that activates
+      // all the component strategies which in turn sets up listeners, we only want to
+      // re-set up resources and when we have
+      // released them in ReleaseResources.
+
+
+      if (this._IsCustomElement() && this._hammerIsUnregistered) {
+        var messaging = this._getComponentMessaging();
+
+        messaging.setupResources();
+        this._hammerIsUnregistered = false;
+      }
+    },
+
     /**
      * If we have asynchronous converter loading, the input is readonly and a loading indicator
      * is shown to the user.
@@ -7750,8 +7793,7 @@ define(['exports', 'ojs/ojcore-base', 'ojs/ojcomponentcore', 'jquery', 'ojs/ojla
     jqLauncher[0].removeEventListener('change', this._eatChangeAndClickOnPress, true);
 
     if (DomUtils.isTouchSupported()) {
-      jqLauncher.ojHammer().off('press');
-      jqLauncher.ojHammer('destroy');
+      this.releaseResources();
       jqLauncher.off('contextmenu', this._eatContextMenuOnOpenPopupListener);
       this._eatContextMenuOnOpenPopupListener = null;
       this._inPressEvent = null;
@@ -7770,50 +7812,36 @@ define(['exports', 'ojs/ojcore-base', 'ojs/ojcomponentcore', 'jquery', 'ojs/ojla
   PopupMessagingStrategy.prototype._registerLauncherEvents = function () {
     var closeEvents;
     var closePopupCallback;
-    var compDefaults;
-    var events;
-    var hammerOptions;
     var jqLauncher = this.GetLauncher();
     var nonPressOpenEvents;
-    var openPopupCallback;
-    var pressEventIndex;
-    compDefaults = PopupMessagingStrategy._DEFAULTS_BY_COMPONENT[this.GetComponent().widgetName];
-    events = compDefaults ? compDefaults.events : PopupMessagingStrategy._DEFAULTS_BY_COMPONENT.default.events; // 1. associate the ojPopup component to wrapper <div> for popup content
+
+    var events = this._getPopupEventNamesToRegisterForComp(); // 1. associate the ojPopup component to wrapper <div> for popup content
     // 2. wire up on() event handlers for registered events that open and close popup. E.g., focusin.
     // 3. autoDismissal happens automatically when focus leaves component. For other events like
     // mouseover it's required to call off()
 
-    if (events.open) {
-      openPopupCallback = this._openPopupCallback;
 
-      if (!openPopupCallback) {
-        openPopupCallback = this._openPopup.bind(this);
-        this._openPopupCallback = openPopupCallback;
+    if (events.open) {
+      if (!this._openPopupCallback) {
+        this._openPopupCallback = this._openPopup.bind(this);
       } // separate out press event, namespace the events string, and attach event handler
 
 
-      pressEventIndex = events.open.indexOf('press');
+      var needsPressEvent = this._needsPressEvent(events);
+
       nonPressOpenEvents = this._getNamespacedEvents(events.open.replace('press', ''), PopupMessagingStrategy._OPEN_NAMESPACE);
-      jqLauncher.on(nonPressOpenEvents, openPopupCallback); // The pressHold gesture also fires a contextmenu event on Windows 10 touch.
+      jqLauncher.on(nonPressOpenEvents, this._openPopupCallback); // The pressHold gesture also fires a contextmenu event on Windows 10 touch.
       // Prevent that here for components that use 'press' for popup messaging as
       // the context menu causes the popup message window to close. Note that this
       // means the context menu will be disabled for these components.
 
-      if (DomUtils.isTouchSupported() && pressEventIndex !== -1) {
+      if (DomUtils.isTouchSupported() && needsPressEvent) {
         this._eatContextMenuOnOpenPopupListener = function () {
           return false;
         };
 
-        jqLauncher.on('contextmenu', this._eatContextMenuOnOpenPopupListener); // for radios and checkboxes, on ios, press hold brings up popup, but release closes it
-        // and checks it, so in this case we have to eat the click/change events. this happens
-        // in the openPopupCallback
-
-        hammerOptions = {
-          recognizers: [[Hammer.Press, {
-            time: 750
-          }]]
-        };
-        jqLauncher.ojHammer(hammerOptions).on('press', openPopupCallback);
+        jqLauncher.on('contextmenu', this._eatContextMenuOnOpenPopupListener);
+        this.registerHammerOpenCallbackOnLauncher();
       }
     }
 
@@ -7829,6 +7857,73 @@ define(['exports', 'ojs/ojcore-base', 'ojs/ojcomponentcore', 'jquery', 'ojs/ojla
       jqLauncher.on(closeEvents, closePopupCallback);
     }
   };
+  /**
+   * Symmetrical method to releaseResources. Sets up resources that get
+   * removed in releaseResources.
+   * @private
+   * @memberof oj.PopupMessagingStrategy
+   * @instance
+   */
+
+
+  PopupMessagingStrategy.prototype.setupResources = function () {
+    var events = this._getPopupEventNamesToRegisterForComp();
+
+    var needsPressEvent = this._needsPressEvent(events);
+
+    if (DomUtils.isTouchSupported() && needsPressEvent) {
+      this.registerHammerOpenCallbackOnLauncher();
+    }
+  };
+  /**
+   * Register Hammer Press Event. Make sure you are on touch and you
+   * need the press event before calling this function.
+   * @private
+   * @memberof oj.PopupMessagingStrategy
+   * @instance
+   */
+
+
+  PopupMessagingStrategy.prototype.registerHammerOpenCallbackOnLauncher = function () {
+    var jqLauncher = this.GetLauncher();
+
+    if (jqLauncher == null) {
+      return;
+    } // for radios and checkboxes, on ios, press hold brings up popup, but release closes it
+    // and checks it, so in this case we have to eat the click/change events. this happens
+    // in the openPopupCallback
+
+
+    var hammerOptions = {
+      recognizers: [[Hammer.Press, {
+        time: 750
+      }]]
+    };
+    jqLauncher.ojHammer(hammerOptions).on('press', this._openPopupCallback);
+  };
+  /**
+  * Releases resources that would otherwise leak memory if they were not released
+  * when the component's dom is removed.
+  * For example, Hammer events are put on the document and will result in
+  * detached dom memory leak if not removed.
+  * @private
+  * @memberof oj.PopupMessagingStrategy
+  * @instance
+  */
+
+
+  PopupMessagingStrategy.prototype.releaseResources = function () {
+    PopupMessagingStrategy.superclass.update.call(this);
+
+    var events = this._getPopupEventNamesToRegisterForComp();
+
+    var needsPressEvent = this._needsPressEvent(events);
+
+    if (DomUtils.isTouchSupported() && needsPressEvent) {
+      var jqLauncher = this.GetLauncher();
+      jqLauncher.off('press').ojHammer('destroy');
+    }
+  };
   /**
    * Turn the events string into an array, add namespace, and turn it back into a string.
    * @param {string} events e.g., "focusin mousedown"
@@ -7903,6 +7998,36 @@ define(['exports', 'ojs/ojcore-base', 'ojs/ojcomponentcore', 'jquery', 'ojs/ojla
     };
     return popupPositionOptions;
   };
+  /**
+   * Returns the events
+   *
+   * @private
+   * @memberof oj.PopupMessagingStrategy
+   * @instance
+   */
+
+
+  PopupMessagingStrategy.prototype._getPopupEventNamesToRegisterForComp = function () {
+    var compDefaults = PopupMessagingStrategy._DEFAULTS_BY_COMPONENT[this.GetComponent().widgetName];
+
+    return compDefaults ? compDefaults.events : PopupMessagingStrategy._DEFAULTS_BY_COMPONENT.default.events;
+  };
+  /**
+   * Returns true if it needs a press event.
+   *
+   * @private
+   * @memberof oj.PopupMessagingStrategy
+   * @instance
+   */
+
+
+  PopupMessagingStrategy.prototype._needsPressEvent = function (events) {
+    if (events && events.open) {
+      return events.open.indexOf('press') !== -1;
+    }
+
+    return false;
+  };
   /**
    * Returns a jquery element that a messaging popup is bound to.
    *

package/dist/js/libs/oj/debug_es5/ojjquery-hammer.js

@@ -36,6 +36,8 @@ define(['ojs/ojlogger', 'hammerjs', 'jquery'], function (Logger, Hammer, $) {
               hammer.destroy();
               $el.removeData('ojHammer');
             }
+
+            $el.off('remove.ojHammer');
           });
 
         default:
@@ -44,7 +46,13 @@ define(['ojs/ojlogger', 'hammerjs', 'jquery'], function (Logger, Hammer, $) {
             var $el = $(this);
 
             if (!$el.data('ojHammer')) {
-              $el.data('ojHammer', new Hammer.Manager($el[0], options));
+              var mgr = new Hammer.Manager($el[0], options);
+              $el.on('remove.ojHammer', function (evt) {
+                if (evt.target === $el[0]) {
+                  mgr.destroy();
+                }
+              });
+              $el.data('ojHammer', mgr);
             }
           });
       }