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

Some demo issues patches #6

Closed
wants to merge 3 commits into from
Closed

Some demo issues patches #6

wants to merge 3 commits into from

Conversation

mattfenwick
Copy link

Based on this issue.

@nbubna nbubna mentioned this pull request Aug 23, 2013
nbubna added a commit that referenced this pull request Aug 23, 2013
nbubna added a commit that referenced this pull request Aug 23, 2013
@nbubna
Copy link
Owner

nbubna commented Aug 23, 2013

Ok, i didn't merge the PR straight-up, but i think i captured the spirit of most of the fixes you sent. I may still do some tweaking to the demo controls/instructions, but things have improved much. Thanks for the fixes!!!

@nbubna nbubna closed this Aug 23, 2013
@mattfenwick
Copy link
Author

Looks like your patches captured the spirit of my intentions better than mine did. Thanks for taking the time to consider this.

@nbubna
Copy link
Owner

nbubna commented Aug 26, 2013

Thanks for pointing it out and the patch to get me started!

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.

2 participants