Fixed MaterialCollectionItem prevents clicking on materialLink #720

Author: kevzlou7979

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

@@ -65,8 +65,10 @@ protected void onLoad() {
             // Stop propagation of event when checkbox / other elements has
             // been clicked to avoid duplicate events.
             if (Element.as(event.getNativeEvent().getEventTarget()) != getElement()) {
-                event.stopPropagation();
-                event.preventDefault();
+                if (getType() == CollectionType.CHECKBOX) {
+                    event.stopPropagation();
+                    event.preventDefault();
+                }
             }
             for (Widget w : MaterialCollectionItem.this) {
                 if (w instanceof MaterialCollectionSecondary) {

gwt-material/src/test/java/gwt/material/design/client/ui/MaterialCollectionTest.java

@@ -107,12 +107,20 @@ public void testCollectionItem() {
         // when / then
         for (int i = 1; i <= 5; i++) {
             MaterialCollectionItem item = new MaterialCollectionItem();
+            boolean[] linkClickHandler = {false};
             boolean[] clickHandler = {false};
+
+            MaterialLink link = new MaterialLink("Link");
+            link.addClickHandler(clickEvent -> linkClickHandler[0] = true);
             item.addClickHandler(clickEvent -> clickHandler[0] = true);
-            fireClickEvent(item);
-            assertTrue(clickHandler[0]);
+            item.add(link);
             collection.add(item);
             assertNotNull(item);
+
+            fireClickEvent(item);
+            fireClickEvent(link);
+            assertTrue(clickHandler[0]);
+            assertTrue(linkClickHandler[0]);
         }
         assertEquals(5, collection.getChildren().size());
     }