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

Makes semantic.gs work with the latest stylus #66

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

eugeneware
Copy link

The grid.styl file was quite broken in the current release. It also didn't generate the same CSS as the less examples, and didn't support push and pull.

I've updated grid.styl to work properly, support push and pull, generate the same CSS as less, and also provided example stylus sheets for all the standard examples.

@scottwhitworth
Copy link

Thanks for the fix... working great!

@Delapouite
Copy link

Great. Thanks for this fixes Eugene.

@csmosx
Copy link

csmosx commented Jul 31, 2012

+1 I probably should've checked the pull requests before fixing all the same things.

@aredridel
Copy link

+1!

@sergeevabc
Copy link

@nharbour, thank you!
I hope your commits will be merged someday.

@nijikokun
Copy link

I've added in support for omega (see neat.burboun.io for more information as to how useful it is, sidebars etc):

https://gist.github.com/Nijikokun/6144380

Also, lots of issues if you don't use brackets on column and I couldn't figure out how to do row, added columns support to push and pull as well.

@abhoopathy
Copy link

@eugeneware, thanks for doing this! I've used it on a couple projects so far.

One thing I noted was I had to rename the 'push' mixin. Stylus has a native push method, which is used by nib to generate vendor prefixes:
https://github.com/visionmedia/nib/blob/master/lib/nib/vendor.styl#L147
http://learnboost.github.io/stylus/docs/bifs.html

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.

8 participants