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

Use faces #10

Closed
wants to merge 1 commit into from
Closed

Use faces #10

wants to merge 1 commit into from

Conversation

quozl
Copy link
Collaborator

@quozl quozl commented May 26, 2016

  • discard circle, rectangle, and triangle shapes,
  • show the player with a face instead of a simple circle,
  • show local bonus players with eyes facing toward extra controls on XO laptops.

@quozl
Copy link
Collaborator Author

quozl commented May 26, 2016

screenshot of maze activity

@godiard
Copy link
Contributor

godiard commented May 26, 2016

Not sure about this change. The difference between players is too subtle (also eyes direction can be interpreted as movement direction). And maybe is better use a image file with the face instead of draw it with cairo.
Just my humble opinion, I gave you commit access to the repository.

@quozl
Copy link
Collaborator Author

quozl commented May 26, 2016

Thanks. Yes, I struggled with that. Different colours are used when collaborating over network.

The use case is local keyboard without collaboration. On the one hand, children are generally excellent at identifying direction of gaze, and any practice is just fun. This arises from infant neural networks, and becomes part of the mirror neural response. For those with undeveloped mirror neural response the game will be that little bit more challenging.

I'll rework it to use monochrome for the local bonus players;

  • black background with white ring,
  • grey background with black ring.

An interesting hidden use case is to activate the local bonus players while in a network collaboration game; they don't appear on the other displays, so can be used as markers.

- discard circle, rectangle, and triangle shapes,

- show the player with a face instead of a simple circle,

- show local bonus players with eyes facing toward extra controls on XO
  laptops, without colour.
@quozl
Copy link
Collaborator Author

quozl commented May 27, 2016

Thanks for the feedback. Change made, force pushed, please review. If okay, I'll push without merge.

I've chosen not to use black background, but instead two shades of grey that are difficult to differentiate when the players are a long way from each other. My guess is that differentiation, without looking at the eyes, is really only needed when the players are in close proximity in the maze. Screenshot of close proximity in simplest maze:

screenshot of _maze activity _

@godiard
Copy link
Contributor

godiard commented May 27, 2016

Feel free to push :)

@quozl
Copy link
Collaborator Author

quozl commented May 29, 2016

Was pushed as 0d745ba.

@quozl quozl closed this May 29, 2016
@quozl quozl deleted the 2016-147-faces branch May 29, 2016 21:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants