From 69dce9f88d412d60a136b77084a5ea43bb416a0b Mon Sep 17 00:00:00 2001 From: Victor Woeltjen Date: Fri, 22 May 2015 13:25:39 -0700 Subject: [PATCH] [Info Bubble] Add clarifying comments Add clarifying comments for the info gesture, WTD-884. --- .../inspect/src/gestures/InfoGesture.js | 17 +++++++++++++++++ 1 file changed, 17 insertions(+) diff --git a/platform/commonUI/inspect/src/gestures/InfoGesture.js b/platform/commonUI/inspect/src/gestures/InfoGesture.js index 584d45d40c..e79c0b6c75 100644 --- a/platform/commonUI/inspect/src/gestures/InfoGesture.js +++ b/platform/commonUI/inspect/src/gestures/InfoGesture.js @@ -5,6 +5,18 @@ define( function () { "use strict"; + /** + * The `info` gesture displays domain object metadata in a + * bubble on hover. + * + * @constructor + * @param $timeout Angular's `$timeout` + * @param {InfoService} infoService a service which shows info bubbles + * @param {number} DELAY delay, in milliseconds, before bubble appears + * @param element jqLite-wrapped DOM element + * @param {DomainObject} domainObject the domain object for which to + * show information + */ function InfoGesture($timeout, infoService, DELAY, element, domainObject) { var dismissBubble, pendingBubble, @@ -65,6 +77,11 @@ define( scopeOff = element.scope().$on('$destroy', hideBubble); return { + /** + * Detach any event handlers associated with this gesture. + * @memberof InfoGesture + * @method + */ destroy: function () { // Dismiss any active bubble... hideBubble();