Skip to content

Conversation

@davidowens
Copy link

Following from the issue/PR from @mightyplow
Issue #24
PR #25

@MrSauceman
Copy link

Can we please get this merged?

@jessepinho
Copy link

Do we need both container and contentWrapper properties?

@davidowens
Copy link
Author

My change deliberately doesn’t change the markup for people who don’t opt in.

I agree that ideally we would reduce the number of wrappers, but that could potentially be a breaking change

@antoniojl
Copy link

container=false might also be a nice (non-breaking) addition

@misstricky
Copy link

Could this please be merged into master?

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.

5 participants