Skip to content
This repository has been archived by the owner on Nov 22, 2023. It is now read-only.

Use bevy_mod_raycast to handle tile selection #262

Merged
merged 17 commits into from
Feb 3, 2023
Merged

Use bevy_mod_raycast to handle tile selection #262

merged 17 commits into from
Feb 3, 2023

Conversation

alice-i-cecile
Copy link
Contributor

@alice-i-cecile alice-i-cecile commented Feb 3, 2023 •
edited
Loading

Progress towards #257.

Status

  • update selected tiles
  • tile selection logic
  • debug visualization of selected tiles
  • smarter strategy for displaying selected tiles: should not clobber display of terrain type (punted to Selected tiles should be displayed more clearly #263)
  • smarter material caching
  • fix input bug: Shift + Left Click seems to somehow be clobbering all other tile select inputs. Changing the keybindings resolves the problem, and selection logic has been tested.

@alice-i-cecile alice-i-cecile added enhancement input Mice and keyboards and gamepads player interaction The tools the player uses to interact with the game world graphics making art real labels Feb 3, 2023
@alice-i-cecile alice-i-cecile mentioned this pull request Feb 3, 2023
Closed
@alice-i-cecile alice-i-cecile changed the title Use bevy_mod_picking to handle tile selection Use bevy_mod_raycast to handle tile selection Feb 3, 2023
@alice-i-cecile alice-i-cecile marked this pull request as ready for review February 3, 2023 17:20
@alice-i-cecile alice-i-cecile enabled auto-merge (squash) February 3, 2023 17:22
@alice-i-cecile alice-i-cecile merged commit dc33e76 into main Feb 3, 2023
@alice-i-cecile alice-i-cecile deleted the pick-me branch February 3, 2023 17:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
graphics making art real input Mice and keyboards and gamepads player interaction The tools the player uses to interact with the game world
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@alice-i-cecile