Resolves #40 duplicate call to toggle in PaperItem

javier-godoy · javier-godoy · commit c7c974d38141 · 2019-11-12T10:01:49.000-03:00
diff --git a/src/main/java/com/flowingcode/addons/applayout/PaperItem.java b/src/main/java/com/flowingcode/addons/applayout/PaperItem.java
@@ -60,8 +60,6 @@ public PaperItem(String title, Command command, AppDrawer appDrawer) {
 			this.getElement().addEventListener("click", e->{
 				command.execute();
 				Optional.ofNullable(appDrawer).ifPresent(AppDrawer::toggle);
-				if (appDrawer!=null)
-					appDrawer.getElement().executeJs("this.toggle()");
 			});
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -60,8 +60,6 @@ public PaperItem(String title, Command command, AppDrawer appDrawer) {`
`60`	`60`	`this.getElement().addEventListener("click", e->{`
`61`	`61`	`command.execute();`
`62`	`62`	`Optional.ofNullable(appDrawer).ifPresent(AppDrawer::toggle);`
`63`		`- if (appDrawer!=null)`
`64`		`- appDrawer.getElement().executeJs("this.toggle()");`
`65`	`63`	`});`
`66`	`64`	`}`
`67`	`65`	`}`