From 672b5d415d880188bf503eb7c333a2bb76028eeb Mon Sep 17 00:00:00 2001 From: vinh123456789 Date: Sat, 30 Jul 2016 20:55:56 +0700 Subject: [PATCH 1/2] Fix issue #4275 Fix issue #4275, when using useLabels: false, dropdown("clear") won't remove class "active" from "item" --- src/definitions/modules/dropdown.js | 1 + 1 file changed, 1 insertion(+) diff --git a/src/definitions/modules/dropdown.js b/src/definitions/modules/dropdown.js index 53e22e4e9b..7f4c9b8867 100644 --- a/src/definitions/modules/dropdown.js +++ b/src/definitions/modules/dropdown.js @@ -2076,6 +2076,7 @@ $.fn.dropdown = function(parameters) { clear: function() { if(module.is.multiple()) { module.remove.labels(); + module.remove.activeItem(); } else { module.remove.activeItem(); From f28c1403752792faed793dc3127e037cb79534a1 Mon Sep 17 00:00:00 2001 From: vinh123456789 Date: Sun, 31 Jul 2016 08:00:18 +0700 Subject: [PATCH 2/2] Move module.remove.activeItem() to outside of if else statements --- src/definitions/modules/dropdown.js | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/src/definitions/modules/dropdown.js b/src/definitions/modules/dropdown.js index 7f4c9b8867..4ad56eeb30 100644 --- a/src/definitions/modules/dropdown.js +++ b/src/definitions/modules/dropdown.js @@ -2075,13 +2075,12 @@ $.fn.dropdown = function(parameters) { clear: function() { if(module.is.multiple()) { - module.remove.labels(); - module.remove.activeItem(); + module.remove.labels(); } - else { - module.remove.activeItem(); + else { module.remove.selectedItem(); } + module.remove.activeItem(); module.set.placeholderText(); module.clearValue(); },