Minimap kann vergrößert werden #16
Conversation
# Conflicts: # frontend/src/ts/Player.ts # frontend/src/ts/backend/GameMapWithBackend.ts # frontend/src/ts/gameObjects/_GameObject.ts
r-zander
left a comment
r-zander
left a comment
There was a problem hiding this comment.
Paar kleine Sachen noch, aber sonst einwandfrei.
| min-width: 1px; //dummy width to render and read height | ||
|
|
||
| background-color: @backgroundColorDark; | ||
| background-clip: content-box; // don't let the background mix with the border |
There was a problem hiding this comment.
Die große Map hat ja keinen Border, also kann die Zeile weg
|
|
||
| function update() { | ||
| this.trackedGameObjects.forEach(gameObject => { | ||
| // TODO Rotation des largeMapIcons anpassen |
There was a problem hiding this comment.
Dieses TODO wurde erledigt und kann weg
| if (Game.state !== Game.States.PLAYING) { | ||
| return; | ||
| } | ||
| if (event) { |
There was a problem hiding this comment.
Unnötig. Das prüft einfach nur, ob es ein Event gibt - was immer der Fall ist. Auch der parameter event kann dann weg.
| this.scale = this.width / this.mapWidth; | ||
| this.iconSizeFactor = this.scale * sizeFactorRelatedToMapSize; | ||
|
|
||
| this.domElement = document.getElementById("largeMap"); |
There was a problem hiding this comment.
Schau mal, oben der container ist bereits genau dasselbe Element. Schieb mal diese Zeile nach Zeile 47 und ändere sie zu this.domElement = container
# Conflicts: # frontend/src/ts/Game.ts # frontend/src/ts/Player.ts
|
@xyckno Nicht wundern, ich habe 'develop' einmal in deinen Branch gemerged, damit es keine Merge Conflicts mehr gibt. Unter anderem gibt es jetzt type safe events und interfaces für Game :) |
https://trello.com/c/c6TmNfOS/341-minimap-kann-vergr%C3%B6%C3%9Fert-werden