SWUI V2 #418

Merged
Chaoscaot merged 11 commits from swui-v2 into main 2026-06-18 14:10:59 +02:00
Owner

Signed-off-by: Chaoscaot max@maxsp.de

Signed-off-by: Chaoscaot <max@maxsp.de>
Chaoscaot added 1 commit 2026-06-10 00:37:10 +02:00
SWUI V2
Pull Request Build / Build (pull_request) Successful in 1m19s
5ce69e6b7a
Signed-off-by: Chaoscaot <max@maxsp.de>
YoyoNow reviewed 2026-06-10 08:59:17 +02:00
YoyoNow reviewed 2026-06-10 09:03:19 +02:00
YoyoNow left a comment
Owner

Sieht erstmal ok aus. Fehlt halt noch ein bisschen utility aber ansonsten ist es ok

Sieht erstmal ok aus. Fehlt halt noch ein bisschen utility aber ansonsten ist es ok
@@ -0,0 +69,4 @@
onClose()
InventoryClickEvent.getHandlerList().unregister(this)
Owner

Hier bitte generisches unregister. Sodass man nicht vergiss irgendwas zu unregistern (HandlerList.unregisterAll oder ähnliches)

Hier bitte generisches unregister. Sodass man nicht vergiss irgendwas zu unregistern (`HandlerList.unregisterAll` oder ähnliches)
Chaoscaot marked this conversation as resolved
@@ -0,0 +37,4 @@
render()
}
var item: ItemStack = ItemStack.of(Material.AIR)
Owner

Hier fehlt natürlich noch das man das Item per hand zusammenbaut, also name ist ne property, material, amount etc.

Hier fehlt natürlich noch das man das Item per hand zusammenbaut, also name ist ne property, material, amount etc.
Chaoscaot marked this conversation as resolved
Chaoscaot added 1 commit 2026-06-10 09:28:01 +02:00
Merge branch 'main' into swui-v2
Pull Request Build / Build (pull_request) Successful in 1m35s
105beaf7a6
Chaoscaot added 3 commits 2026-06-10 09:28:28 +02:00
D4rkr34lm requested changes 2026-06-10 20:43:24 +02:00
Dismissed
@@ -0,0 +70,4 @@
@EventHandler
fun onInventoryClick(event: InventoryClickEvent) {
if (event.inventory.holder is UIWindow) {
val window = event.inventory.holder as UIWindow
Owner

val holder = event.inventory.holder; if(holder is UIWindow) { ...

wäre hier deutlich schöner

` val holder = event.inventory.holder; if(holder is UIWindow) { ... ` wäre hier deutlich schöner
Chaoscaot marked this conversation as resolved
YoyoNow requested changes 2026-06-10 20:47:49 +02:00
Dismissed
@@ -39,2 +39,4 @@
dependsOn(":SchematicSystem:shadowJar")
dependsOn(":KotlinCore:shadowJar")
template = "Bau21"
worldName = "73"
Owner

Den musst du auch in eine local steamwar.properties Datei legen im BauSystem Projekt. Dies wird so nicht mehr in main möglich sein!

Den musst du auch in eine local steamwar.properties Datei legen im BauSystem Projekt. Dies wird so nicht mehr in main möglich sein!
Chaoscaot marked this conversation as resolved
Chaoscaot added 1 commit 2026-06-10 21:23:59 +02:00
Add Top-Level State and more Utils
Pull Request Build / Build (pull_request) Successful in 1m21s
a6681ffaf6
Signed-off-by: Chaoscaot <max@maxsp.de>
Chaoscaot added 1 commit 2026-06-10 21:43:07 +02:00
Cleanup for Merge
Pull Request Build / Build (pull_request) Successful in 1m40s
8050f51424
Signed-off-by: Chaoscaot <max@maxsp.de>
Chaoscaot added 1 commit 2026-06-10 21:43:26 +02:00
Merge branch 'main' into swui-v2
Pull Request Build / Build (pull_request) Successful in 1m54s
997292d58e
Chaoscaot marked the pull request as ready for review 2026-06-10 21:43:28 +02:00
Chaoscaot requested review from D4rkr34lm 2026-06-11 23:52:18 +02:00
Chaoscaot requested review from YoyoNow 2026-06-11 23:52:19 +02:00
YoyoNow requested changes 2026-06-12 11:16:23 +02:00
Dismissed
@@ -34,6 +34,7 @@ dependencies {
compileOnly(libs.classindex)
annotationProcessor(libs.classindex)
compileOnly(project(":SpigotCore", "default"))
compileOnly(project(":KotlinCore", "default"))
Owner

Aktuell doch noch nicht benötigt oder?

Aktuell doch noch nicht benötigt oder?
@@ -37,5 +37,6 @@ tasks.register<DevServer>("DevBau21") {
dependsOn(":SpigotCore:shadowJar")
dependsOn(":BauSystem:shadowJar")
dependsOn(":SchematicSystem:shadowJar")
dependsOn(":KotlinCore:shadowJar")
Owner

Aktuell doch noch nicht benötigt oder?

Aktuell doch noch nicht benötigt oder?
@@ -1,3 +1,5 @@
import org.jetbrains.kotlin.gradle.tasks.KotlinCompile
Owner

Der Import wirklich nötig?

Der Import wirklich nötig?
Author
Owner

ja.

ja.
YoyoNow marked this conversation as resolved
@@ -0,0 +22,4 @@
import kotlin.properties.ReadWriteProperty
import kotlin.reflect.KProperty
class Observer<T>(private var value: T): ReadWriteProperty<Any?, T> {
Owner

Observer chaining? Mit sowas wie .transform auf einem Observer womit du einen DelegatedObserver bekommt der nur ein getValue hat?

Observer chaining? Mit sowas wie .transform auf einem Observer womit du einen DelegatedObserver bekommt der nur ein getValue hat?
Chaoscaot marked this conversation as resolved
@@ -0,0 +45,4 @@
bukkitInv = KotlinCore.plugin.server.createInventory(this, size * 9, Component.translatable(title))
}
constructor(type: InventoryType, title: String, player: Player, render: WindowContext.() -> Unit): this(player, render) {
Owner

Hier vllt prüfen, dass man Chest nicht nutzen darf!

Hier vllt prüfen, dass man Chest nicht nutzen darf!
Chaoscaot marked this conversation as resolved
@@ -0,0 +33,4 @@
}
override fun resetSlot(x: Int, y: Int) {
window.bukkitInv.setItem(x + y * 9, ItemStack.of(Material.AIR))
Owner

x + y * 9 vllt noch in eine variable packen?

`x + y * 9` vllt noch in eine variable packen?
Chaoscaot marked this conversation as resolved
Chaoscaot added 1 commit 2026-06-12 12:15:35 +02:00
D4rkr34lm requested changes 2026-06-17 21:29:47 +02:00
Dismissed
@@ -0,0 +41,4 @@
return this
}
fun listen(callback: (T) -> Unit): () -> Unit {
Owner

Sollte den callback on init callen

Sollte den callback on init callen
D4rkr34lm marked this conversation as resolved
@@ -0,0 +58,4 @@
}
}
class Observer<T>(private var value: T): ReadWriteProperty<Any?, T>, Observable<T>() {
Owner

Warum hält der observer einen value, sollte warscheinlic von observable geowned werden

Warum hält der observer einen value, sollte warscheinlic von observable geowned werden
D4rkr34lm marked this conversation as resolved
@@ -0,0 +19,4 @@
package de.steamwar.kotlin.ui
abstract class ObserverListener {
Owner

die ganze struktur scheint mir hier überkomplieziert, ich würde eher erwarten Observer -> Observable und mehr nicht

die ganze struktur scheint mir hier überkomplieziert, ich würde eher erwarten Observer -> Observable und mehr nicht
D4rkr34lm marked this conversation as resolved
Chaoscaot added 1 commit 2026-06-17 21:49:47 +02:00
Fix Observer
Pull Request Build / Build (pull_request) Successful in 1m59s
7af586ec16
Signed-off-by: Chaoscaot <max@maxsp.de>
Chaoscaot added 1 commit 2026-06-17 21:50:03 +02:00
Merge branch 'main' into swui-v2
Pull Request Build / Build (pull_request) Successful in 2m8s
Backport CommonCore / Create CommonCore backport PRs (pull_request) Successful in 9s
7038dc2aa5
D4rkr34lm approved these changes 2026-06-17 21:51:24 +02:00
YoyoNow approved these changes 2026-06-17 21:51:33 +02:00
Chaoscaot merged commit b68a27fbb6 into main 2026-06-18 14:10:59 +02:00
Chaoscaot deleted branch swui-v2 2026-06-18 14:10:59 +02:00
Sign in to join this conversation.