Fixed MaterialCollapsibleItem.setActive(boolean) not working #719

Author: kevzlou7979

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

@@ -106,7 +106,6 @@ public boolean remove(Widget w) {
     public void expand() {
         if (body != null) {
             setActive(true);
-            body.setDisplay(Display.BLOCK);
         }
     }
 
@@ -116,7 +115,6 @@ public void expand() {
     public void collapse() {
         if (body != null) {
             setActive(false);
-            body.setDisplay(Display.NONE);
         }
     }
 
@@ -160,6 +158,10 @@ public void setActive(boolean active) {
                 header.addStyleName(CssName.ACTIVE);
             }
         }
+
+        if (body != null) {
+            body.setDisplay(active ? Display.BLOCK : Display.NONE);
+        }
     }
 
     @Override

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

@@ -197,11 +197,13 @@ public void testPreSelection() {
             assertTrue(item.isActive());
             assertTrue(item.getElement().hasClassName(CssName.ACTIVE));
             assertTrue(item.getHeader().getElement().hasClassName(CssName.ACTIVE));
+            assertEquals(Display.BLOCK.getCssName(), item.getBody().getElement().getStyle().getDisplay());
 
             item.setActive(false);
             assertFalse(item.isActive());
             assertFalse(item.getElement().hasClassName(CssName.ACTIVE));
             assertFalse(item.getHeader().getElement().hasClassName(CssName.ACTIVE));
+            assertEquals(Display.NONE.getCssName(), item.getBody().getElement().getStyle().getDisplay());
         }
 
         collapsible.clearActive();