Skip to content

Conversation

@brysgo
Copy link

@brysgo brysgo commented Jan 17, 2018

Hey!

Thanks for writing this example, to get it working I updated a few dependencies. For some reason it is off center (and off screen on smaller screens) because of the update. Haven't had time to debug it, but I figure I'll leave it here.

Cheers,
Bryan

@JeffreyATW
Copy link
Owner

Was it not working with its existing dependencies?

@brysgo
Copy link
Author

brysgo commented Jan 17, 2018

@JeffreyATW - It only worked when I updated react-scripts. The rest I updated after as a bonus. When I just updated react scripts, things were properly aligned. That is how I know the update messed it up. It mostly works though.

@JeffreyATW
Copy link
Owner

It's very likely the misalignment was caused by upgrading react-game-kit, which had a major version bump. If you'd like to work further on making this code compatible with the latest version then I can leave this PR open for updates sometime later.

@brysgo
Copy link
Author

brysgo commented Jan 17, 2018

@JeffreyATW - leave it open, I will probably fix it.

@JeffreyATW JeffreyATW self-requested a review January 17, 2018 18:07
Copy link
Owner

@JeffreyATW JeffreyATW left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awaiting stage realignment after version bumps

@@ -1 +1 @@
Testing out [create-react-app](https://github.com/facebookincubator/create-react-app) and [react-game-kit](https://github.com/FormidableLabs/react-game-kit).
T esting out [create-react-app](https://github.com/facebookincubator/create-react-app) and [react-game-kit](https://github.com/FormidableLabs/react-game-kit).
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo here

@brysgo
Copy link
Author

brysgo commented Jan 17, 2018

I was reading through the commit history of react-game-kit, so I thought we didn't need to monkey patch, but I added it back when I realize it was reverted.

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