Fixed #297 & #278

Major clean up on MaterialAutocomplete with recent PR merged.

Author: kevzlou7979

src/main/java/gwt/material/design/addins/client/autocomplete/MaterialAutoComplete.java

@@ -219,19 +219,25 @@ public MaterialAutoComplete(SuggestOracle suggestions) {
         setup(suggestions);
     }
 
+    private HandlerRegistration listHandler, itemBoxKeyDownHandler, itemBoxBlurHandler, itemBoxClickHandler;
+
     @Override
     protected void onLoad() {
         super.onLoad();
 
-        registerHandler(list.addDomHandler(event -> suggestBox.showSuggestionList(), ClickEvent.getType()));
+        loadHandlers();
+    }
+
+    protected void loadHandlers() {
+        listHandler = list.addDomHandler(event -> suggestBox.showSuggestionList(), ClickEvent.getType());
 
-        registerHandler(itemBox.addBlurHandler(blurEvent -> {
+        itemBoxBlurHandler = itemBox.addBlurHandler(blurEvent -> {
             if (getValue().size() > 0) {
                 label.addStyleName(CssName.ACTIVE);
             }
-        }));
+        });
 
-        registerHandler(itemBox.addKeyDownHandler(event -> {
+        itemBoxKeyDownHandler = itemBox.addKeyDownHandler(event -> {
             boolean changed = false;
 
             switch (event.getNativeKeyCode()) {
@@ -278,34 +284,52 @@ protected void onLoad() {
                     itemBox.setFocus(true);
                     break;
             }
+        });
 
-            if (changed) {
-                ValueChangeEvent.fire(MaterialAutoComplete.this, getValue());
-            }
-        }));
+        itemBoxClickHandler = itemBox.addClickHandler(event -> suggestBox.showSuggestionList());
+    }
 
-        registerHandler(itemBox.addClickHandler(event -> suggestBox.showSuggestionList()));
+    @Override
+    protected void onUnload() {
+        super.onUnload();
 
-        registerHandler(suggestBox.addSelectionHandler(selectionEvent -> {
-            Suggestion selectedItem = selectionEvent.getSelectedItem();
-            itemBox.setValue("");
-            if (addItem(selectedItem)) {
-                ValueChangeEvent.fire(MaterialAutoComplete.this, getValue());
-            }
-            itemBox.setFocus(true);
-        }));
+        unloadHandlers();
+    }
+
+    protected void unloadHandlers() {
+        removeHandler(listHandler);
+        removeHandler(itemBoxBlurHandler);
+        removeHandler(itemBoxKeyDownHandler);
+        removeHandler(itemBoxClickHandler);
     }
 
     /**
      * Generate and build the List Items to be set on Auto Complete box.
      */
     protected void setup(SuggestOracle suggestions) {
+
+        if (itemBoxKeyDownHandler != null) {
+            itemBoxKeyDownHandler.removeHandler();
+        }
+
         list.setStyleName(AddinsCssName.MULTIVALUESUGGESTBOX_LIST);
         this.suggestions = suggestions;
         final ListItem item = new ListItem();
 
         item.setStyleName(AddinsCssName.MULTIVALUESUGGESTBOX_INPUT_TOKEN);
+
         suggestBox = new SuggestBox(suggestions, itemBox);
+        suggestBox.addSelectionHandler(selectionEvent -> {
+            Suggestion selectedItem = selectionEvent.getSelectedItem();
+            itemBox.setValue("");
+            if (addItem(selectedItem)) {
+                ValueChangeEvent.fire(MaterialAutoComplete.this, getValue());
+            }
+            itemBox.setFocus(true);
+        });
+
+        loadHandlers();
+
         setLimit(this.limit);
         String autocompleteId = DOM.createUniqueId();
         itemBox.getElement().setId(autocompleteId);
@@ -316,7 +340,7 @@ protected void setup(SuggestOracle suggestions) {
 
         panel.add(list);
         panel.getElement().setAttribute("onclick",
-            "document.getElementById('" + autocompleteId + "').focus()");
+                "document.getElementById('" + autocompleteId + "').focus()");
         panel.add(errorLabel);
         suggestBox.setFocus(true);
     }
@@ -526,6 +550,7 @@ public void setLimit(int limit) {
      * Set the number of suggestions to be displayed to the user. This differs from
      * setLimit() which set both the suggestions displayed AND the limit of values
      * allowed within the autocomplete.
+     *
      * @param limit
      */
     public void setAutoSuggestLimit(int limit) {
@@ -545,9 +570,8 @@ public void setPlaceholder(String placeholder) {
     }
 
     /**
-     * @see gwt.material.design.client.ui.MaterialValueBox#setLabel(String)
-     *
      * @param label
+     * @see gwt.material.design.client.ui.MaterialValueBox#setLabel(String)
      */
     public void setLabel(String label) {
         this.label.setText(label);