Skip to content

feat: ✨ add fow handeling#75

Merged
ChrisKrane merged 3 commits intomainfrom
feature/74-add-fog-of-war
Apr 20, 2025
Merged

feat: ✨ add fow handeling#75
ChrisKrane merged 3 commits intomainfrom
feature/74-add-fog-of-war

Conversation

@ChrisKrane
Copy link
Copy Markdown
Collaborator

@ChrisKrane ChrisKrane commented Apr 20, 2025

Description

Adds Fog of War to the game

Related Issues

Closes: #74

Changes Made

  • Add Fog of War component
  • Make rendering dependent on the Fog of War game setting
  • Clear fog 1 tile around pieces

Screenshots (if applicable)

image

image

Checklist

Check applicable items

  • My code follows the style guidelines of this project.
  • I have performed a self-review of my own code.
  • I have commented on my code where necessary.
  • I have added tests that prove my fix is effective or that my feature works.
  • New and existing unit tests pass locally with my changes.

@ChrisKrane ChrisKrane added enhancement New feature or request ready for review PR ready for reviewing priority - high labels Apr 20, 2025
@ChrisKrane ChrisKrane self-assigned this Apr 20, 2025
@ChrisKrane ChrisKrane linked an issue Apr 20, 2025 that may be closed by this pull request
[Feature] Add fog of war #74
Closed
1 task
michaelbrusegard
michaelbrusegard previously requested changes Apr 20, 2025
View reviewed changes
Comment thread chessevolved_model/src/main/kotlin/io/github/chessevolved/systems/MovementSystem.kt Outdated
Comment thread chessevolved_presenter/src/main/kotlin/io/github/chessevolved/presenters/GamePresenter.kt
Comment thread chessevolved_presenter/src/main/kotlin/io/github/chessevolved/presenters/GamePresenter.kt Outdated
@CraZyB1336 CraZyB1336 dismissed michaelbrusegard’s stale review April 20, 2025 12:54

He finished, and we don't want to wait for reapproval

@ChrisKrane ChrisKrane merged commit b9def04 into main Apr 20, 2025
1 check passed
@ChrisKrane ChrisKrane deleted the feature/74-add-fog-of-war branch April 20, 2025 12:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@michaelbrusegard michaelbrusegard michaelbrusegard left review comments

@CraZyB1336 CraZyB1336 CraZyB1336 approved these changes

@sivertun sivertun Awaiting requested review from sivertun

Assignees

@ChrisKrane ChrisKrane

Labels

enhancement New feature or request priority - high ready for review PR ready for reviewing

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Feature] Add fog of war

3 participants

@ChrisKrane @michaelbrusegard @CraZyB1336