From 2555a7a4aa2dc5b73c2ea408247bc07d5e4c3e02 Mon Sep 17 00:00:00 2001 From: Alexander Ebert Date: Sun, 17 Jul 2016 18:02:45 +0200 Subject: [PATCH] Code cleanup --- .../files/js/WoltLab/WCF/Ui/Dropdown/Simple.js | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/wcfsetup/install/files/js/WoltLab/WCF/Ui/Dropdown/Simple.js b/wcfsetup/install/files/js/WoltLab/WCF/Ui/Dropdown/Simple.js index 4cda14648c..6adad84ff7 100644 --- a/wcfsetup/install/files/js/WoltLab/WCF/Ui/Dropdown/Simple.js +++ b/wcfsetup/install/files/js/WoltLab/WCF/Ui/Dropdown/Simple.js @@ -1,8 +1,8 @@ /** - * Simple Dropdown + * Simple dropdown implementation. * * @author Alexander Ebert - * @copyright 2001-2015 WoltLab GmbH + * @copyright 2001-2016 WoltLab GmbH * @license GNU Lesser General Public License * @module WoltLab/WCF/Ui/Dropdown/Simple */ @@ -279,6 +279,7 @@ define( */ _onDialogScroll: function(event) { var dialogContent = event.currentTarget; + //noinspection JSCheckFunctionSignatures var dropdowns = elBySelAll('.dropdown.dropdownOpen', dialogContent); for (var i = 0, length = dropdowns.length; i < length; i++) { @@ -342,7 +343,7 @@ define( * Toggles the dropdown's state between open and close. * * @param {?Event} event event object, should be 'null' if targetId is given - * @param {string=} targetId dropdown wrapper id + * @param {string?} targetId dropdown wrapper id * @param {Element=} alternateElement alternative reference element for alignment * @return {boolean} 'false' if event is not null */ @@ -351,6 +352,7 @@ define( event.preventDefault(); event.stopPropagation(); + //noinspection JSCheckFunctionSignatures targetId = elData(event.currentTarget, 'target'); } @@ -397,8 +399,8 @@ define( } }).bind(this)); - // TODO - WCF.Dropdown.Interactive.Handler.closeAll(); + //noinspection JSDeprecatedSymbols + window.WCF.Dropdown.Interactive.Handler.closeAll(); return (event === null); } -- 2.20.1