From 860bb6d61581409ffc042bc66e521243a8208e70 Mon Sep 17 00:00:00 2001 From: Pierre Goiffon Date: Mon, 8 Oct 2018 17:56:34 +0200 Subject: [PATCH] =?UTF-8?q?N=C2=B0931=20AttributeTagSet=20widget=20js=20wr?= =?UTF-8?q?apper=20:=20fix=20remaining=20console.debug=20call,=20and=20fix?= =?UTF-8?q?=20indentations?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- js/jquery.itop-set-widget.js | 36 +++++++++++++++++++----------------- 1 file changed, 19 insertions(+), 17 deletions(-) diff --git a/js/jquery.itop-set-widget.js b/js/jquery.itop-set-widget.js index 8627ca0d2b..b9e31c0005 100644 --- a/js/jquery.itop-set-widget.js +++ b/js/jquery.itop-set-widget.js @@ -166,13 +166,15 @@ $.widget('itop.set_widget', _bindEvents: function($widgetElement) { var setWidget = this; $widgetElement.bind("update", function() { - console.debug("update event in Selectize !", this); + if (setWidget.options.isDebug) { + console.debug("update event in Selectize !", this); + } var $this = $(this); if ($this.prop("disabled")) { - setWidget.disable(); - } else { - setWidget.enable(); - } + setWidget.disable(); + } else { + setWidget.enable(); + } }); }, @@ -212,20 +214,20 @@ $.widget('itop.set_widget', }, /** - *

Updating selection widget : - *

* - * @param inputWidget Selectize object - * @private - */ + *

For partial codes at first I was thinking about using the Selectize render callback, but it is called before onItemAdd/onItemRemove :(
+ * Indeed as we only need to have partial items on first display, this callback is the right place O:) + * + * @param inputWidget Selectize object + * @private + */ _onInitialize: function (inputWidget) { var setWidget = this; if (this.options.isDebug) {