Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Blockly: Disable Keyboard and Mouse #1797

Merged
merged 2 commits into from
Mar 17, 2025
Merged

Blockly: Disable Keyboard and Mouse #1797

merged 2 commits into from
Mar 17, 2025

Conversation

Tobi511
Copy link
Contributor

@Tobi511 Tobi511 commented Mar 17, 2025

Ich habe das PlayerSystem innerhalb der onSetup() in der client.java entfernt, um alle Steuerungen über Maus und Tastatur zu deaktivieren.
Zusätzlich habe ich auf Wunsch von @Flamtky den Keybind des BlocklyHUDs in der createHero() entfernt, da es nicht mehr verwendet wird.

closes #1747

@Tobi511 Tobi511 requested a review from AMatutat March 17, 2025 13:09
@Tobi511 Tobi511 self-assigned this Mar 17, 2025
@Tobi511 Tobi511 requested a review from cagix as a code owner March 17, 2025 13:09
@AMatutat
Copy link
Contributor

AMatutat commented Mar 17, 2025

Frage: Im Dungeon wird das PlayerComponent genutzt um den Helden zu identifizieren.

Aus ECSManagement

  public static Optional<Entity> hero() {
    return entityStream().filter(e -> e.isPresent(PlayerComponent.class)).findFirst();
  }

Geht das jetzt nicht kaputt? Ich glaube das Pathfinding der Monster baut z.B darauf auf.

@Flamtky
Copy link
Member

Flamtky commented Mar 17, 2025

Frage: Im Dungeon wird das PlayerComponent genutzt um den Helden zu identifizieren.

Aus ECSManagement

  public static Optional<Entity> hero() {
    return entityStream().filter(e -> e.isPresent(PlayerComponent.class)).findFirst();
  }

Geht das jetzt nicht kaputt? Ich glaube das Pathfinding der Monster baut z.B darauf auf.

Haben es getestet. Der Held behält sein PlayerComponent

@AMatutat
Copy link
Contributor

Frage: Im Dungeon wird das PlayerComponent genutzt um den Helden zu identifizieren.
Aus ECSManagement

  public static Optional<Entity> hero() {
    return entityStream().filter(e -> e.isPresent(PlayerComponent.class)).findFirst();
  }

Geht das jetzt nicht kaputt? Ich glaube das Pathfinding der Monster baut z.B darauf auf.

Haben es getestet. Der Held behält sein PlayerComponent

ach mist, kann nicht lesen. Das SYSTEM ist raus...

mb, sorry

Copy link
Contributor

@AMatutat AMatutat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hab fragen

Co-authored-by: Andre Matutat <32962412+AMatutat@users.noreply.github.com>
Copy link
Contributor

@AMatutat AMatutat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good job

@AMatutat AMatutat merged commit 30a1326 into master Mar 17, 2025
9 checks passed
@AMatutat AMatutat deleted the tobi511/keyboard-mouse branch March 17, 2025 14:13
@cagix
Copy link
Member

cagix commented Mar 17, 2025

puh. das blockly-hud sollte doch eigentlich längst draußen sein? haben wir wohl übersehen.

@AMatutat
Copy link
Contributor

AMatutat commented Mar 17, 2025

puh. das blockly-hud sollte doch eigentlich längst draußen sein? haben wir wohl übersehen.

Ist es wohl, siehe #1797 (comment)
ich wusste davon nur noch nichts und hier war etwas Deadcode verblieben (und jetzt entfernt) :P

@cagix
Copy link
Member

cagix commented Mar 17, 2025

puh. das blockly-hud sollte doch eigentlich längst draußen sein? haben wir wohl übersehen.

Ist es wohl, siehe #1797 (comment) ich wusste davon nur noch nichts und hier war etwas Deadcode verblieben (und jetzt entfernt) :P

ja, eben. die stelle haben wir beim aufräumen übersehen. gut, dass das hier mit aufgefallen ist.

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.

Blockly: Disable Keyboard and Mouse
4 participants