Skip to content

Ogranicz natywne dialogi do wprowadzania tekstu - #21

Merged
WieszczY85 merged 1 commit into
mainfrom
syntaxdevteam-improvements/naprawic-wyswietlanie-dialogu-przy-rozszerzaniu-dziaki
Sep 21, 2026
Merged

WieszczY85 merged 1 commit into
mainfrom
syntaxdevteam-improvements/naprawic-wyswietlanie-dialogu-przy-rozszerzaniu-dziaki

Conversation

@WieszczY85

Copy link
Copy Markdown
Member

Motivation

  • Zaobserwowano niespójne przełączanie między natywnym dialogiem a GUI przy rozszerzaniu działki; interfejs powinien używać natywnego dialogu tylko tam, gdzie wymagane jest wprowadzenie tekstu (np. zmiana nazwy).

Description

  • Ograniczono adapter Paper Dialog tylko do formularzy tekstowych przez wprowadzenie showTextInput i użycie go wyłącznie w ścieżce zmiany nazwy (PlotInteractions, PaperDialogBackend).
  • Przywrócono i wymuszono stałe GUI ekwipunkowe dla potwierdzeń i wyborów bez pola tekstowego w operacjach takich jak tworzenie, usuwanie i rozszerzanie działek oraz zarządzanie członkami (ChunkExpandGUI, ClaimConfirmGUI, UnclaimConfirmGUI, MembersGUI).
  • Usunięto/wyłączyłem backend wyborów i potwierdzeń oparty na natywnych dialogach oraz zaktualizowano komunikaty i konfigurację, aby jednoznacznie dokumentować, że natywny dialog służy wyłącznie do wprowadzania tekstu (config.yml, docs/dialogs.md, docs/wiki/Panel-dzialki.md, pliki językowe).
  • Uprzątnięto powiązany, nieużywany kod i uproszczono logikę sesji/dialogów tak, by zachować kompatybilny fallback do GUI ekwipunkowego i czatu tam, gdzie to konieczne.

Testing

  • Uruchomiono chmod +x gradlew oraz ./gradlew test --console=plain, które zakończyły się sukcesem i testy jednostkowe przeszły pomyślnie (BUILD SUCCESSFUL).
  • Sprawdzono brak problemów stylu i błędów przez git diff --check, bez wykrytych błędów.

Codex Task

@WieszczY85
WieszczY85 merged commit 33fca31 into main Sep 21, 2026
1 check passed
@WieszczY85
WieszczY85 deleted the syntaxdevteam-improvements/naprawic-wyswietlanie-dialogu-przy-rozszerzaniu-dziaki branch September 21, 2026 07:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant