Support for "Open All" buttons (#14)

Co-authored-by: mohammad <[email protected]>
AlahmadiQ8 · Mar 13, 2023 · 26dbf29 · 26dbf29
1 parent 0b21d45
commit 26dbf29
Show file tree

Hide file tree

Showing 2 changed files with 61 additions and 23 deletions.
diff --git a/src/main/kotlin/com/nbk/weyay/weyaydesktopclient/MainController.kt b/src/main/kotlin/com/nbk/weyay/weyaydesktopclient/MainController.kt
@@ -11,6 +11,7 @@ import javafx.event.EventHandler
 import javafx.fxml.FXML
 import javafx.fxml.FXMLLoader
 import javafx.fxml.Initializable
+import javafx.scene.Node
 import javafx.scene.control.*
 import javafx.scene.layout.*
 import javafx.scene.paint.Color
@@ -70,32 +71,35 @@ class MainController : CoroutineScope, Initializable {
             saveButton.isDisable = newTab?.userData == null
             saveAsButton.isDisable = newTab?.userData == null
         }
-        NodeFactory(recentFilesContainer.children, RecentFiles.recentItems) { path ->
-            val pathParsed = Path.of(path)
-            BorderPane().apply {
-                left = Hyperlink(pathParsed.fileName.toString()).apply {
-                    setOnAction {
-                        openFile(Path.of(path).toFile())
-                    }
 
-                }
-                right = Hyperlink().apply {
-                    graphic = FontIcon("fas-times").apply { iconColor = Color.gray(0.0, 0.25) }
-                    setOnAction {
-                        RecentFiles.removeRecentFile(path)
+        NodeFactory(
+            recentFilesContainer.children,
+            RecentFiles.recentItems,
+            ::recentFilesHomeScreenBuilder,
+            listOf(
+                BorderPane().apply {
+                    left = Hyperlink("Open All").apply {
+                        setOnAction {
+                            RecentFiles.recentItems.forEach { openFile(Path.of(it).toFile()) }
+                        }
                     }
                 }
-            }
-        }
+            )
+        )
 
-        NodeFactory(recentFilesMenu.items, RecentFiles.recentItems) { path ->
-            val pathParsed = Path.of(path)
-            MenuItem(pathParsed.fileName.toString()).apply {
-                setOnAction {
-                    openFile(Path.of(path).toFile())
+        NodeFactory(
+            recentFilesMenu.items,
+            RecentFiles.recentItems,
+            ::recentFilesMenuItemBuilder,
+            listOf(
+                SeparatorMenuItem(),
+                MenuItem("Open All").apply {
+                    setOnAction {
+                        RecentFiles.recentItems.forEach { openFile(Path.of(it).toFile()) }
+                    }
                 }
-            }
-        }
+            )
+        )
 
         recentFilesMenu.disableProperty().bind(Bindings.isEmpty(RecentFiles.recentItems))
         recentFilesLabel.textProperty().bind(
@@ -280,4 +284,31 @@ class MainController : CoroutineScope, Initializable {
         }
         return tableData
     }
+
+    fun recentFilesHomeScreenBuilder(path: String): Node {
+        val pathParsed = Path.of(path)
+        return BorderPane().apply {
+            left = Hyperlink(pathParsed.fileName.toString()).apply {
+                setOnAction {
+                    openFile(Path.of(path).toFile())
+                }
+
+            }
+            right = Hyperlink().apply {
+                graphic = FontIcon("fas-times").apply { iconColor = Color.gray(0.0, 0.25) }
+                setOnAction {
+                    RecentFiles.removeRecentFile(path)
+                }
+            }
+        }
+    }
+
+    fun recentFilesMenuItemBuilder(path: String): MenuItem {
+        val pathParsed = Path.of(path)
+        return MenuItem(pathParsed.fileName.toString()).apply {
+            setOnAction {
+                openFile(Path.of(path).toFile())
+            }
+        }
+    }
 }
diff --git a/src/main/kotlin/com/nbk/weyay/weyaydesktopclient/NodeFactory.kt b/src/main/kotlin/com/nbk/weyay/weyaydesktopclient/NodeFactory.kt
@@ -6,7 +6,8 @@ import javafx.collections.ObservableList
 class NodeFactory<T, R>(
     private val nodes: ObservableList<R>,
     private val data: ObservableList<T>,
-    private val buildNodeFor: (model: T) -> R
+    private val buildNodeFor: (model: T) -> R,
+    private val additionalNodes: Collection<R> = emptyList()
 ) {
     init {
         data.addListener(ListChangeListener { ch ->
@@ -26,13 +27,19 @@ class NodeFactory<T, R>(
                         nodes[index] = node
                     }
                 } else if (ch.wasRemoved()) {
-                    nodes.remove(ch.from, ch.from + ch.removedSize)
+                    if (nodes.size > 0) nodes.remove(ch.from, ch.from + ch.removedSize)
+                    if (nodes.size == 1) nodes.clear()
                 } else if (ch.wasAdded()) {
+                    val initialSize = nodes.size
                     nodes.addAll(ch.from, buildNodesFor(ch.addedSubList))
+                    if (initialSize == 0) {
+                        nodes.addAll(additionalNodes)
+                    }
                 }
             }
         })
         rebuildAllNodes()
+
     }
 
     private fun buildNodesFor(dataIn: List<T>): List<R> {