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

Better end turn #239

Merged
merged 2 commits into from May 27, 2014
Merged

Better end turn #239

merged 2 commits into from May 27, 2014

Conversation

michaeljb
Copy link
Collaborator

Replaces Goko's "End Turn" button with one in the Salvager sidebar, should reduce the chance of ending your turn by misclick.

Also includes a JSLint setting tweak so that everything passes JSLint again.

Tested in latest Chrome and Firefox on OS X Mavericks.

@aiannacc
Copy link
Owner

I'm confused. I don't see any change.
endturn

Testing on Chromium 33 on Arch Linux.

@michaeljb
Copy link
Collaborator Author

Did you check "Better End Turn button" in the settings dialog? It's under the "Game" tab.

@aiannacc
Copy link
Owner

Duh. Okay. Great work!

aiannacc added a commit that referenced this pull request May 27, 2014
@aiannacc aiannacc merged commit 5d87bbd into aiannacc:beta May 27, 2014
@michaeljb michaeljb deleted the better-end-turn branch May 27, 2014 03:48
@michaeljb
Copy link
Collaborator Author

Thanks! 👍

@theblankman
Copy link
Collaborator

Just tried this on Chrome 35, OSX 10.9.3, and I was seeing multiples of the new "End Turn" button. They all worked, there was just a row of them, one for each game I played with the option on, like so:
screen shot 2014-05-27 at 1 58 28 am
I added a comment to the commit where I think the issue is.

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

3 participants