Skip to content

Conversation

@mkarnicki
Copy link
Contributor

@mkarnicki mkarnicki commented Nov 2, 2017

First of all, thank you for the amazing work guys. I've been waiting for the premier for a while now and I'm crazy excited about it! Going through the docs and intend to put this into use ASAP.

Secondly, I didn't know if it's okay to PR against master or I should have created a feature branch (not sure what's the best practice for this project) so I apologise for potential mistake :).

Lastly:

The initial form of the comment seemed to claim that main content area is wide enough to apply md: styles between 768px and 868px, which is not what you guys meant. I'm not a native speaker, but I think I've cleared up the wording, because the sentence seemed to contradict itself. Of course, you can still improve my change with even better wording.

I understand that the example shown says:
Apply sm from 500px upwards.
Apply md from 668px to 767px.
Revert to sm here.
Apply md from 868px upwards.

Thanks!

@adamwathan
Copy link
Member

That's definitely clearer, thank you!

@adamwathan adamwathan merged commit cf18cb2 into tailwindlabs:master Nov 2, 2017
DCzajkowski pushed a commit to DCzajkowski/tailwindcss that referenced this pull request Jul 23, 2019
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