Implement some bulding construction prerequisites #140
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This serves as a preparation for handling of different mouse buttons. It also subtly changes behavior: the events are always fully consumed thus multiple clicks of the same button within one frame are handled as a single click.
This makes it possible to access entities inserted by the controller from other systems (in other crates) in the Update stage.
The systems were executed twice in a quick succession, on press and on release.
It is the intention to have all input even handling on one place, that is inside command.rs. Systems in this module send events to other systems which implement low-level implementation. With this slightly updated architecture, it will be easier to overload left-click handling (e.g. selection vs construction of a new building). Relates to #23.
This has the following benefits: * Future object spawning crates (e.g. controller) won't have to depend on map crate and structures which are tight with persistent map format. * Spawn events no longer need to be buffered and resend. Relates to #23.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.