Jonathan/Babushka
2
0
Fork 0
Code Pull Requests 1 Releases Activity

Fight rework premature merge #22

Merged
Jonathan merged 16 commits from feature/fight_rework into develop 2025-11-04 10:25:14 +01:00
Conversation 10 Commits 16 Files Changed 108
+5088 -791
Jonathan commented 2025-10-29 05:39:14 +01:00
Owner
Copy Link
Copy Source

A working but not finished version of the new fight system.

Includes:

  • The entire fight logic is based on a state machine outlined on Miro
  • Procedurally generated fight world. Currently only generating two rooms next to each other.
  • Basic fight attack

Known bugs:

  • Dead enemies still attack
  • No background in fight
  • Buttons other than attack do not work
  • Enemy groups do not disappear when beaten
  • probably more...
A working but not finished version of the new fight system. Includes: - The entire fight logic is based on a state machine outlined on [Miro](https://miro.com/app/board/uXjVK8YEprM=/?moveToWidget=3458764640805655262&cot=14) - Procedurally generated fight world. Currently only generating two rooms next to each other. - Basic fight attack Known bugs: - Dead enemies still attack - No background in fight - Buttons other than attack do not work - Enemy groups do not disappear when beaten - probably more...
Jonathan added 14 commits 2025-10-29 05:39:15 +01:00
Bootstrap fight system 2929c5dd25 
- Fight World data structure
- Generating basic fight world
- Opening correct fight room
- Block paths in fight rooms
- Transition between rooms
Fight happening base setup daabcdc5ee
Made fight fightable 8055381478
Added basic action animation 52f1560c48
Added basic health bar f0db6372e5
Added blob attack action f2bf44bf9e
cap health at minimum 0 3f23f4471b
Added minigame 96df233496
Added minigame to attack action ff8e4a85d2
✨Added minigame label for each region 7d35c991f3
✨Added theme-based region colors 224863f891
🐛Exiting fight happening when one side wins bd6432f568
♻️Code cleanup 719b1fa800
♻️ Implemented FightersFormation to keep track of the fight entered state of the fighters 45d4b26b6a
Jonathan marked the pull request as work in progress 2025-10-29 05:39:29 +01:00
Jonathan marked the pull request as ready for review 2025-10-29 05:39:46 +01:00
Jonathan requested review from kziolkowski 2025-10-29 05:39:53 +01:00
kziolkowski was assigned by Jonathan 2025-10-29 05:39:59 +01:00
kziolkowski reviewed 2025-10-30 22:45:34 +01:00
scripts/CSharp/Common/CharacterControls/InteractionArea2D.cs Outdated
@@ -13,2 +15,2 @@
[Export] private bool _useSprite = true;
[Export] private CanvasItem _spriteToOutline;
//[Export] private bool _useSprite = true;
[Export(PropertyHint.ArrayType)] private CanvasItem[] _spriteToOutline = [];
kziolkowski commented 2025-10-30 22:45:34 +01:00
Owner
Copy Link
Copy Source

Bitte in Plural umbenennen.

Bitte in Plural umbenennen.
Jonathan marked this conversation as resolved
kziolkowski reviewed 2025-10-30 22:47:01 +01:00
scripts/CSharp/Common/Fight/ActionAnimationController.cs Outdated
@@ -0,0 +10,4 @@
#endregion
[Export] private AllFightersVisual _allFightersVisual = null!;
kziolkowski commented 2025-10-30 22:47:01 +01:00
Owner
Copy Link
Copy Source

Nur aus Neugier, weil ich das noch nie gesehen hab: Was macht diese "null!" Zuweisung hier?

Nur aus Neugier, weil ich das noch nie gesehen hab: Was macht diese "null!" Zuweisung hier?
Jonathan marked this conversation as resolved
kziolkowski reviewed 2025-10-30 22:48:25 +01:00
scripts/CSharp/Common/Fight/ActionDetails/MinigameActionDetail.cs Outdated
@@ -0,0 +1,26 @@
using System.Collections.Generic;
kziolkowski commented 2025-10-30 22:48:25 +01:00
Owner
Copy Link
Copy Source

Ich weiß nicht, was es mit diesem Unicode-Character hier auf sich hat, aber ich gehe einfach mal davon aus, dass er uns keine Ärger machen wird 👀

Ich weiß nicht, was es mit diesem Unicode-Character hier auf sich hat, aber ich gehe einfach mal davon aus, dass er uns keine Ärger machen wird 👀
Jonathan marked this conversation as resolved
kziolkowski reviewed 2025-10-30 22:49:12 +01:00
scripts/CSharp/Common/Fight/ActionDetails/MinigameActionDetail.cs Outdated
@@ -0,0 +1,26 @@
using System.Collections.Generic;
namespace Babushka.scripts.CSharp.Common.Fight.ActionDetails;
kziolkowski commented 2025-10-30 22:49:12 +01:00
Owner
Copy Link
Copy Source

Vielleicht macht es Sinn, das Fightsystem in einen eigenen Namespace-Bereich auszulagern und nicht einfach mit unter "Common" abzulegen?

Vielleicht macht es Sinn, das Fightsystem in einen eigenen Namespace-Bereich auszulagern und nicht einfach mit unter "Common" abzulegen?
Jonathan marked this conversation as resolved
kziolkowski reviewed 2025-10-30 22:49:24 +01:00
scripts/CSharp/Common/Fight/ActionDetails/MinigameActionDetail.cs Outdated
@@ -0,0 +4,4 @@
public class MinigameActionDetail : FighterAction.FighterActionDetail
{
// settings
kziolkowski commented 2025-10-30 22:49:24 +01:00
Owner
Copy Link
Copy Source

brauchen wir den Kommentar noch?

brauchen wir den Kommentar noch?
Jonathan marked this conversation as resolved
kziolkowski reviewed 2025-10-30 22:50:59 +01:00
scripts/CSharp/Common/Fight/ActionDetails/TargetSelectActionDetail.cs Outdated
@@ -0,0 +6,4 @@
{
public enum VisualRange
{
Single
kziolkowski commented 2025-10-30 22:50:59 +01:00
Owner
Copy Link
Copy Source

Ein Enum mit nur einem Eintrag? Was soll da voraussichtlich noch kommen? Vielleicht

  • Single und Multiple oder vielleicht
  • Single, Double, Triple usw.?
Ein Enum mit nur einem Eintrag? Was soll da voraussichtlich noch kommen? Vielleicht - Single und Multiple oder vielleicht - Single, Double, Triple usw.?
Jonathan marked this conversation as resolved
kziolkowski reviewed 2025-10-30 22:53:29 +01:00
scripts/CSharp/Common/Fight/ActionDetails/TargetSelectActionDetail.cs Outdated
@@ -0,0 +34,4 @@
public FightWorld.Fighter GetTarget()
{
return target ?? throw new InvalidOperationException("No target selected");
kziolkowski commented 2025-10-30 22:53:29 +01:00
Owner
Copy Link
Copy Source

Muss das immer ne Exception werfen? Also kann es keinen Fall geben, wo es keine Selection gibt? ISt es dann für die Praxis nicht vielleicht nicer, wenn man nen Default definiert? Oder passiert das auf den höheren Detailebenen?

Muss das immer ne Exception werfen? Also kann es keinen Fall geben, wo es keine Selection gibt? ISt es dann für die Praxis nicht vielleicht nicer, wenn man nen Default definiert? Oder passiert das auf den höheren Detailebenen?
Jonathan commented 2025-11-04 10:22:22 +01:00
Author
Owner
Copy Link
Copy Source

GetTarget sollte nur im ActionExecute und ActionAnimate state aufgerufen werden. Die states werden nur erreicht, nach dem alle action details completed sind. Das heißt es muss ein target gesetzt worden sein.

GetTarget sollte nur im ActionExecute und ActionAnimate state aufgerufen werden. Die states werden nur erreicht, nach dem alle action details completed sind. Das heißt es muss ein target gesetzt worden sein.
Jonathan marked this conversation as resolved
kziolkowski reviewed 2025-10-30 22:58:18 +01:00
scripts/CSharp/Common/Fight/Actions/BlobAttackAction.cs Outdated
@@ -0,0 +4,4 @@
namespace Babushka.scripts.CSharp.Common.Fight.Actions;
public class BlobAttackAction : FighterAction
kziolkowski commented 2025-10-30 22:58:18 +01:00
Owner
Copy Link
Copy Source

Verstehe ich das richtig, dass wir für jeden Gegner(typ) ne neue Klasse anlegen müssen? Falls ja: Wäre es vielleicht auch möglich, das über Config-Ressourcen ganz oder teilwese konfigurierbar zu machen?

Verstehe ich das richtig, dass wir für jeden Gegner(typ) ne neue Klasse anlegen müssen? Falls ja: Wäre es vielleicht auch möglich, das über Config-Ressourcen ganz oder teilwese konfigurierbar zu machen?
Jonathan commented 2025-11-04 10:24:53 +01:00
Author
Owner
Copy Link
Copy Source

Es braucht nicht jeder gegner type eine klasse, sondern jede action. Es können auch mehrere gegner typen die gleiche action haben.

Das irgendwie cool konfigurierbar zu machen fänd ich auch sinnvoll, aber erst später und nicht für den prototypen

Es braucht nicht jeder gegner type eine klasse, sondern jede action. Es können auch mehrere gegner typen die gleiche action haben. Das irgendwie cool konfigurierbar zu machen fänd ich auch sinnvoll, aber erst später und nicht für den prototypen
Jonathan marked this conversation as resolved
kziolkowski reviewed 2025-10-30 23:01:50 +01:00
scripts/CSharp/Common/Fight/AllFightersVisual.cs Outdated
@@ -0,0 +7,4 @@
namespace Babushka.scripts.CSharp.Common.Fight;
public partial class AllFightersVisual : Node
kziolkowski commented 2025-10-30 23:01:50 +01:00
Owner
Copy Link
Copy Source

Diese Klasse managed das Layout der Kampfszene, oder? Das Naming hatte mich etwas verwirrt.

Diese Klasse managed das Layout der Kampfszene, oder? Das Naming hatte mich etwas verwirrt.
Jonathan marked this conversation as resolved
kziolkowski approved these changes 2025-11-03 20:20:28 +01:00
kziolkowski left a comment
Owner
Copy Link
Copy Source

please see comments and resolve merge conflicts.

please see comments and resolve merge conflicts.
Jonathan added 1 commit 2025-11-04 05:22:27 +01:00
Live review changes 1edbe7aeb7
Jonathan force-pushed feature/fight_rework from 1edbe7aeb7 to f27f69c15f 2025-11-04 10:07:58 +01:00 Compare
Jonathan merged commit e864c62a3a into develop 2025-11-04 10:25:14 +01:00
Sign in to join this conversation.
Reviewers
No Reviewers
kziolkowski
Labels
No items
No Label
Milestone
No items
No Milestone
Assignees
Clear assignees
ascheck (Anna Scheck) jlink kziolkowski (Katharina Ziolkowski) lkrause Nao Palurdas saderako Sanelschnitte Jonathan
No Assignees kziolkowski
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Jonathan/Babushka#22
Delete Branch "feature/fight_rework"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?