Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Initial Click to Spawn Target #25

Closed
bboudaoud-nv opened this issue Jul 1, 2019 · 4 comments
Closed

Initial Click to Spawn Target #25

bboudaoud-nv opened this issue Jul 1, 2019 · 4 comments
Assignees
Labels
bug Something isn't working invalid This doesn't seem right question Further information is requested

Comments

@bboudaoud-nv
Copy link
Collaborator

When you first open the application (playMode does not matter) and close the tab menu no target is spawned. In order to spawn the first red target you have to left click once. It seems as if this target should just appear without clicking, but I want to verify this before "fixing" the issue.

@bboudaoud-nv bboudaoud-nv added bug Something isn't working invalid This doesn't seem right question Further information is requested labels Jul 1, 2019
@bboudaoud-nv
Copy link
Collaborator Author

Looking a bit deeper here, it seems that this may be intentional. In the Experiment::onInit() we set the m_app->presentationState to PresentationState::initial which seems to just wait for a click to move on to the next state...

@jspjutNV
Copy link
Contributor

Yes, this was an intentional design decision. The text that appears prior to the first target is meant to instruct you how to being the experiment. If it's unclear we should fix that text. If the text blends in too much, maybe we should change the colors.

@bboudaoud-nv
Copy link
Collaborator Author

bboudaoud-nv commented Jul 31, 2019

This makes sense, I strongly suggest we move away from the current text which is

Aim at the target and shoot!

considering it does not prompt the user to click to spawn the first target. Perhaps we should elaborate a bit to something like:

Click to spawn the first target, then aim at the target and shoot!

The biggest thing is that it is clear what the user needs to do to advance the state into spawning the first dummy target.

@bboudaoud-nv
Copy link
Collaborator Author

Since this seems to be more related to how we communicate how to play to the user I am going to close this issue in favor of another issue regarding documenting player instructions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working invalid This doesn't seem right question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants