This fixes #6, adding support for changing icons in toolbar.

Author: mlopezFC

src/main/java/com/flowingcode/addons/applayout/AppToolbar.java

@@ -1,72 +1,74 @@
-package com.flowingcode.addons.applayout;
-
-/*-
- * #%L
- * App Layout Addon
- * %%
- * Copyright (C) 2018 Flowing Code
- * %%
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- * 
- *      http://www.apache.org/licenses/LICENSE-2.0
- * 
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- * #L%
- */
-
-import java.util.ArrayList;
-import java.util.List;
-
-import com.flowingcode.addons.applayout.menu.MenuItem;
-import com.vaadin.flow.component.Component;
-import com.vaadin.flow.component.HasComponents;
-import com.vaadin.flow.component.Tag;
-import com.vaadin.flow.component.dependency.HtmlImport;
-import com.vaadin.flow.component.html.Div;
-
-/**
- * Component that renders the app toolbar
- * 
- * @author mlopez
- *
- */
-@SuppressWarnings("serial")
-@Tag("app-toolbar")
-@HtmlImport("bower_components/app-layout/app-toolbar/app-toolbar.html")
-public class AppToolbar extends Component implements HasComponents {
-	
-	PaperIconButton menu;
-	Div divTitle;
-	
-    public AppToolbar(String title, AppDrawer drawer) {
-    	menu = new PaperIconButton("menu");
-    	menu.getElement().setAttribute("onclick", "" + drawer.getId().get() + ".toggle()");
-    	divTitle = new Div();
-    	divTitle.setText(title);
-    	divTitle.getElement().setAttribute("main-title", true);
-    	this.add(menu,divTitle);
-    }
-
-	public void setToolbarIconButtons(MenuItem[] menuItems) {
-		List<PaperIconButton> toolbarIconButtons = createToolbarIconButtons(menuItems);
-		this.removeAll();
-		this.add(menu,divTitle);
-		toolbarIconButtons.forEach(tib->this.add(tib));
-	}
-
-	private List<PaperIconButton> createToolbarIconButtons(MenuItem[] menuItems) {
-		List<PaperIconButton> result = new ArrayList<>();
-		for (MenuItem menuItem : menuItems) {
-			result.add(new PaperIconButton(menuItem.getIcon(),menuItem.getCommand()));
-		}
-		return result;
-	}
-
-    
-}
+package com.flowingcode.addons.applayout;
+
+/*-
+ * #%L
+ * App Layout Addon
+ * %%
+ * Copyright (C) 2018 Flowing Code
+ * %%
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ * 
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ * 
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ * #L%
+ */
+
+import java.util.ArrayList;
+import java.util.List;
+
+import com.flowingcode.addons.applayout.menu.MenuItem;
+import com.vaadin.flow.component.Component;
+import com.vaadin.flow.component.HasComponents;
+import com.vaadin.flow.component.Tag;
+import com.vaadin.flow.component.dependency.HtmlImport;
+import com.vaadin.flow.component.html.Div;
+
+/**
+ * Component that renders the app toolbar
+ * 
+ * @author mlopez
+ *
+ */
+@SuppressWarnings("serial")
+@Tag("app-toolbar")
+@HtmlImport("bower_components/app-layout/app-toolbar/app-toolbar.html")
+public class AppToolbar extends Component implements HasComponents {
+	
+	PaperIconButton menu;
+	Div divTitle;
+	
+    public AppToolbar(String title, AppDrawer drawer) {
+    	menu = new PaperIconButton("menu");
+    	menu.getElement().setAttribute("onclick", "" + drawer.getId().get() + ".toggle()");
+    	divTitle = new Div();
+    	divTitle.setText(title);
+    	divTitle.getElement().setAttribute("main-title", true);
+    	this.add(menu,divTitle);
+    }
+
+	public void setToolbarIconButtons(MenuItem[] menuItems) {
+		List<PaperIconButton> toolbarIconButtons = createToolbarIconButtons(menuItems);
+		this.removeAll();
+		this.add(menu,divTitle);
+		toolbarIconButtons.forEach(tib->this.add(tib));
+	}
+
+	private List<PaperIconButton> createToolbarIconButtons(MenuItem[] menuItems) {
+        List<PaperIconButton> result = new ArrayList<>();
+        for (MenuItem menuItem : menuItems) {
+                PaperIconButton paperIconButton = new PaperIconButton(menuItem.getIcon(),menuItem.getCommand());
+                menuItem.setRefreshCallback(() -> paperIconButton.setIcon(menuItem.getIcon()));
+                result.add(paperIconButton);
+        }
+        return result;
+}
+
+    
+}