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

Gatsby website example page fixes #1134

Merged
merged 2 commits into from Jun 3, 2019
Merged

Conversation

tgorkin
Copy link
Contributor

@tgorkin tgorkin commented May 31, 2019

Upgrade to ocular-gatsby alpha.35
Update gatsby website example pages to use example config from page context.
Fix styletron dev dependency conflict.
Rename AnimationLoopRunner to AnimationLoopExamplePage based on PR feedback from #1122 .

@tgorkin tgorkin requested a review from ibgreen May 31, 2019 22:17
@coveralls
Copy link

coveralls commented May 31, 2019

Coverage Status

Coverage increased (+1.8%) to 47.829% when pulling eea2e1c on tgorkin/gatsby-example-pages into 8f7ff98 on master.

Copy link
Collaborator

@ibgreen ibgreen left a comment

Choose a reason for hiding this comment

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

Very nice!

canvas: PropTypes.string
};

const defaultProps = {
canvas: 'example-canvas'
canvas: "example-canvas"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Wonder what is happening here, we always used single quotes inside JS code...

@tgorkin tgorkin merged commit 884c17d into master Jun 3, 2019
@tgorkin tgorkin deleted the tgorkin/gatsby-example-pages branch June 3, 2019 19:08
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

4 participants