Skip to content

Conversation

wongmjane
Copy link
Contributor

@wongmjane wongmjane commented Nov 20, 2018

articleLayout.container.flexDirection is will be set to 'column' only when the media is less than 'small'

My #1414 previously broke desktop layout because the flexDirection: column applied to desktop as well, where it shouldn't had because flexDirection: row | initial is required for the sidebar to be placed next to the article

This should now work on both desktop and mobile:

output2

Fixes #1404

@reactjs-bot
Copy link

Deploy preview for reactjs ready!

Built with commit 210b9ad

https://deploy-preview-1416--reactjs.netlify.com

@reactjs-bot
Copy link

Deploy preview for reactjs ready!

Built with commit 918d316

https://deploy-preview-1416--reactjs.netlify.com

@gaearon gaearon merged commit bbfb7c5 into reactjs:master Nov 20, 2018
@gaearon
Copy link
Member

gaearon commented Nov 20, 2018

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants