-
Notifications
You must be signed in to change notification settings - Fork 26.3k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Simplify styled-components example (#5631)
- use `enhanceApp` so that styled-components used in _app.js are server rendered - call parent getInitialProps, fixes #5629 - return `styles`, making the render() method obsolete. cc @mxstbr @probablyup
- Loading branch information
1 parent
a0137d4
commit ed2c379
Showing
1 changed file
with
9 additions
and
18 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,25 +1,16 @@ | ||
import Document, { Head, Main, NextScript } from 'next/document' | ||
import Document from 'next/document' | ||
import { ServerStyleSheet } from 'styled-components' | ||
|
||
export default class MyDocument extends Document { | ||
static getInitialProps ({ renderPage }) { | ||
static async getInitialProps (ctx) { | ||
const sheet = new ServerStyleSheet() | ||
const page = renderPage(App => props => sheet.collectStyles(<App {...props} />)) | ||
const styleTags = sheet.getStyleElement() | ||
return { ...page, styleTags } | ||
} | ||
|
||
render () { | ||
return ( | ||
<html> | ||
<Head> | ||
{this.props.styleTags} | ||
</Head> | ||
<body> | ||
<Main /> | ||
<NextScript /> | ||
</body> | ||
</html> | ||
) | ||
const originalRenderPage = ctx.renderPage | ||
ctx.renderPage = () => originalRenderPage({ | ||
enhanceApp: App => props => sheet.collectStyles(<App {...props} />) | ||
}) | ||
|
||
const initialProps = await Document.getInitialProps(ctx) | ||
return { ...initialProps, styles: [...initialProps.styles, ...sheet.getStyleElement()] } | ||
} | ||
} |
ed2c379
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
ed2c379
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@timneutkens how do I add my own
<html>
and<body>
with this setup?ed2c379
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Like normal, add a
render()
method holding the tags from the documentation