Fixed - MaterialSideNav handles clicks on MaterialSideProfile #351

Author: kevzlou7979

gwt-material/src/main/java/gwt/material/design/client/ui/MaterialSideNav.java

@@ -34,14 +34,16 @@
 import com.google.gwt.user.client.ui.RootPanel;
 import com.google.gwt.user.client.ui.Widget;
 import com.google.web.bindery.event.shared.HandlerRegistration;
-import gwt.material.design.client.base.*;
+import gwt.material.design.client.base.HasSelectables;
+import gwt.material.design.client.base.HasType;
+import gwt.material.design.client.base.HasWaves;
+import gwt.material.design.client.base.MaterialWidget;
 import gwt.material.design.client.base.helper.DOMHelper;
 import gwt.material.design.client.base.mixin.CssTypeMixin;
 import gwt.material.design.client.base.mixin.ToggleStyleMixin;
 import gwt.material.design.client.constants.Edge;
 import gwt.material.design.client.constants.SideNavType;
 import gwt.material.design.client.events.*;
-import gwt.material.design.client.events.ClearActiveEvent.ClearActiveHandler;
 import gwt.material.design.client.events.SideNavClosedEvent.SideNavClosedHandler;
 import gwt.material.design.client.events.SideNavClosingEvent.SideNavClosingHandler;
 import gwt.material.design.client.events.SideNavOpenedEvent.SideNavOpenedHandler;
@@ -160,15 +162,18 @@ public Widget wrap(Widget child) {
             child.getElement().getStyle().setProperty("textAlign", "center");
         }
 
-        boolean collapsible = child instanceof MaterialCollapsible;
-        if(collapsible) {
-            // Since the collapsible is separ
-            ((MaterialCollapsible)child).addClearActiveHandler(new ClearActiveHandler() {
-                @Override
-                public void onClearActive(ClearActiveEvent event) {
-                    clearActive();
+        // Check whether the widget is not selectable by default
+        boolean isNotSelectable = false;
+        if(child instanceof MaterialWidget) {
+            MaterialWidget widget = (MaterialWidget) child;
+            if (widget.getInitialClasses() != null) {
+                if (widget.getInitialClasses().length > 0) {
+                    String initialClass = widget.getInitialClasses()[0];
+                    if(initialClass.contains("side-profile") || initialClass.contains("collapsible")) {
+                        isNotSelectable = true;
+                    }
                 }
-            });
+            }
         }
 
         if(!(child instanceof ListItem)) {
@@ -186,9 +191,9 @@ public void onClearActive(ClearActiveEvent event) {
             child = listItem;
         }
 
-        // Collapsible's should not be selectable
+        // Collapsible and Side Porfile should not be selectable
         final Widget finalChild = child;
-        if(!collapsible) {
+        if(!isNotSelectable) {
             // Active click handler
             finalChild.addDomHandler(new ClickHandler() {
                 @Override