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

MazeWeb: Show Red Face On Start #69

Closed
davelab6 opened this issue Jul 9, 2016 · 6 comments
Closed

MazeWeb: Show Red Face On Start #69

davelab6 opened this issue Jul 9, 2016 · 6 comments
Milestone

Comments

@davelab6
Copy link
Contributor

davelab6 commented Jul 9, 2016

IN http://server.sugarizer.org/activities/MazeWeb.activity/index.html the starting state has a grey circle for the player and a bright flashing color for the exit.

This is confusing because it brings attention to the exit at the start instead of the player.

screen shot 2016-07-09 at 12 11 39

I suggest having the red face of the player rendered in the starting state, rather than having it appear:

screen shot 2016-07-09 at 12 11 46

@SudhanvaMG
Copy link

Hey , i would like to work on this .
Can you let me know how do i proceed ?

@llaske
Copy link
Owner

llaske commented Mar 1, 2017

You had to explore the code of the activity and try to understand where is the face changing.
Send a PR when you've done

@DarkPrince304
Copy link
Contributor

@llaske In case of two player, which players face should we show? Given both are of different color?

Or should the solution be to have the red colour nevertheless?

@rajeevravindran
Copy link
Contributor

How about this? I've set the initial position as a red player.
screenshot_20170313_155006

Becomes gray after the player moves.
screenshot_20170313_155056

@rajeevravindran
Copy link
Contributor

@llaske Worth a pull request?

rajeevravindran added a commit to rajeevravindran/sugarizer that referenced this issue Mar 13, 2017
Draws a red player over the inital position. This draws attention of the user to the inital position.
rajeevravindran added a commit to rajeevravindran/sugarizer that referenced this issue Mar 13, 2017
Draws a red face on the initial position to draw the user's attention
@llaske llaske added this to the v0.9 milestone Mar 28, 2017
@llaske
Copy link
Owner

llaske commented Mar 28, 2017

Fixed in cc8fabe

@llaske llaske closed this as completed Mar 28, 2017
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

No branches or pull requests

5 participants