fix: fix clipping display issues and add layer support by jatoothless · Pull Request #11 · Solvro/web-capybara-escape

jatoothless · 2026-03-09T13:44:47Z

Naprawiłam clippowanie przez podłoge
Lekko poprawiłam rysowanie mapy
Dodałam warstwy

bnszky

Super, właśnie też nad tym się zastanawiałem, żeby przenieść tworzenie mapy do osobnej klasy. Odnośnie zmian z dodawaniem nowym obiektów i refactoringiem zmiany są tutaj: refactor/add-srp-and-ocp. To myślę, że można zmergować

bnszky · 2026-03-16T11:55:43Z

Na mainie są teraz jeszcze kable, więc możesz również je dodać do warst i wtedy rebase

bnszky

Stwórzmy strukturę danych, która pozwoli w łatwy sposób dodawać nowe layery i priorytetować ich kolejność, nazwy powinny znaleźć się w pliku globalnym np constants/global

Tam tworzymy np. listę:
layers = ["background", "floor_decoys", "entities", "wall decoys", "effects"]

automatycznie indeksy oznaczają kolejność i setDepth

Wyobraźmy sobie sytuację, że chcemy dodać nową warstwę np. "entities2" i wiemy, że powinna znaleźć się po "entities". Wtedy w tej liście po prostu wstawimy "entities2" i displayer sam to posortuje.

W kodzie dodamy wtedy tylko add(object, "entities2") i tyle

W tym momencie musielibyśmy zmieniać wiele linijek żeby wprowadzić nową warstwę
CEL: dodawać warstwę tylko w liście layers

I tak dużo już się udało zrobić. Ta zmiana zaoszczędzi później czas i odchudzi jeszcze bardziej kod

bnszky · 2026-04-02T12:56:39Z

client/src/phaser/lib/display.ts

+const TILE_MAPPING: Record<
+  string,
+  { frame: number; isTall?: boolean; frameSecond?: number }
+> = {
+  w1t: { frame: 0, frameSecond: 10, isTall: true },
+  w1: { frame: 0 },
+  w13: { frame: 9 },
+  w2t: { frame: 2, frameSecond: 4, isTall: true },
+  w2: { frame: 2 },
+  w3t: { frame: 3, frameSecond: 4, isTall: true },
+  w3: { frame: 3 },
+  w21: { frame: 8 },
+  f1: { frame: 6 },
+};


Usuń TILE_MAPPING i importuj z constants/blocks. Tam już jest

bnszky · 2026-04-02T13:13:30Z

client/src/phaser/lib/display.ts

+  private BACKGROUND: Phaser.GameObjects.Layer;
+  private FLOOR_DECOYS: Phaser.GameObjects.Layer;
+  private ENTITIES: Phaser.GameObjects.Layer;
+  private WALL_DECOYS: Phaser.GameObjects.Layer;
+  private EFFECTS: Phaser.GameObjects.Layer;


Zamiast tego można napisać
private LAYERS: Phaser.GameObjects.Layer[]

bnszky · 2026-04-02T13:14:19Z

client/src/phaser/lib/display.ts

+    layer:
+      | "background"
+      | "floor decoys"
+      | "entities"
+      | "wall decoys"
+      | "effects",


stwórz interface LayerNames z tymi nowymi nazwami i wtedy wszędzie:

layer: LayerNames

bnszky · 2026-04-02T13:14:35Z

client/src/phaser/lib/display.ts

+    layer: //Layery są tu poukładane od najgłębszego do najpłytszego
+      | "background" //ściany i podłoga
+      | "floor decoys" //wszystko co na podłodze - venty, guziki, kable
+      | "entities" //gracz kapibara i rzeczy które "stoją na podłodze"
+      | "wall decoys" //rzeczy typu drzwi
+      | "effects", //efekty - np dymki


layer: LayerNames

bnszky · 2026-04-02T13:16:15Z

client/src/phaser/scenes/main.ts

+    layer:
+      | "background"
+      | "floor decoys"
+      | "entities"
+      | "wall decoys"
+      | "effects",


bnszky · 2026-04-02T13:24:54Z

i jeszcze zamiast stringów najlepiej zrobić słownik, bo będzie problem jak zmienimy "wall_decoys" na "walls_decoy", więc
np.
LayerNamesEnum = {
BACKGROUND: "background",
etc...
}

add(object, LayerNamesEnum.BACKGROUND)

*Komentarze po angielsku

bnszky

Właśnie o to chodziło, jeszcze tylko dopisałem, żeby nie używać hardkodowanych stringów w kodzie, tylko stworzyć jakiś słownik, gdyby ktoś się pomylił, wpisując np. "entitties" zamiast "entities". Aktualnie linter nam tego nie podświetli z taką literówką, ale kod nie będzie poprawnie działał.

Nie musisz tego poprawiać, mogę to zrobić ja już w osobnym branchu, więc daje approve

bnszky · 2026-04-03T19:32:19Z

client/src/phaser/scenes/main.ts


        for (const crate of message.crates) {
-          this.spawnEntity(this.crates, Crate, crate);
+          this.spawnEntity(this.crates, Crate, crate, "entities");


Suggested change

this.spawnEntity(this.crates, Crate, crate, "entities");

this.spawnEntity(this.crates, Crate, crate, LAYER_NAMES_GLOBAL.ENTITIES);

bnszky · 2026-04-03T19:32:27Z

client/src/phaser/lib/display.ts

+
+        if (tileType !== "") {
+          this.layerMap
+            .get("background")


Suggested change

.get("background")

.get(LAYER_NAMES_GLOBAL.BACKGROUND)

bnszky · 2026-04-03T19:32:33Z

client/src/constants/global.ts

+export const LAYERS = [ //Layers in order of deepest to shallowest
+  "background", //Floor tiles and wall tiles
+  "floor decoys", //Things on the floor such as vents or buttons
+  "entities", //Players, enemies and things above floor decoys like lasers
+  "wall decoys", //Upper parts of walls and doord
+  "effects", //Extra effects rendered on top like speech bubbles
+] as const;


Suggested change

export const LAYERS = [ //Layers in order of deepest to shallowest

"background", //Floor tiles and wall tiles

"floor decoys", //Things on the floor such as vents or buttons

"entities", //Players, enemies and things above floor decoys like lasers

"wall decoys", //Upper parts of walls and doord

"effects", //Extra effects rendered on top like speech bubbles

] as const;

export const LAYER_NAMES_GLOBAL = {

BACKGROUND: "background",

FLOOR_DECOYS: "floor decoys",

ENTITIES: "entities",

WALL_DECOYS: "wall decoys",

EFFECTS: "effects",

}

as const;

export const LAYERS = [

//Layers in order of deepest to shallowest

LAYER_NAMES_GLOBAL.BACKGROUND, //Floor tiles and wall tiles

LAYER_NAMES_GLOBAL.FLOOR_DECOYS, //Things on the floor such as vents or buttons

LAYER_NAMES_GLOBAL.ENTITIES, //Players, enemies and things above floor decoys like lasers

LAYER_NAMES_GLOBAL.WALL_DECOYS, //Upper parts of walls and doord

LAYER_NAMES_GLOBAL.EFFECTS, //Extra effects rendered on top like speech bubbles

] as const;

bnszky · 2026-04-03T19:33:16Z

client/src/phaser/scenes/main.ts

+    layer:
+      | "background"
+      | "floor decoys"
+      | "entities"
+      | "wall decoys"
+      | "effects",


pull-request-size bot added the size/L label Mar 9, 2026

jatoothless requested a review from bnszky March 9, 2026 13:44

jatoothless linked an issue Mar 11, 2026 that may be closed by this pull request

Dodać layery by wyświetlać mapę #10

Open

bnszky assigned jatoothless Mar 15, 2026

bnszky approved these changes Mar 15, 2026

View reviewed changes

jatoothless added 2 commits March 30, 2026 22:16

fix: fix clipping display issues and add layer support

9703d61

fix: fix layering and integrate with recent changes

dc5c535

jatoothless force-pushed the fix/fix-display branch from fdfd652 to dc5c535 Compare April 1, 2026 07:38

bnszky requested changes Apr 2, 2026

View reviewed changes

jatoothless added 2 commits April 2, 2026 21:52

chore: translate comments

670e638

chore: clean up the code

384f39c

jatoothless requested a review from bnszky April 3, 2026 16:14

bnszky approved these changes Apr 3, 2026

View reviewed changes

	this.spawnEntity(this.crates, Crate, crate, "entities");
	this.spawnEntity(this.crates, Crate, crate, LAYER_NAMES_GLOBAL.ENTITIES);

Conversation

jatoothless commented Mar 9, 2026

Uh oh!

bnszky left a comment

Choose a reason for hiding this comment

Uh oh!

bnszky commented Mar 16, 2026

Uh oh!

bnszky left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

bnszky commented Apr 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

bnszky left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

bnszky Apr 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

bnszky commented Apr 2, 2026 •

edited

Loading

bnszky Apr 3, 2026 •

edited

Loading