search

stylus / nib

Stylus mixins, utilities, components, and gradient image generation
http://stylus.github.io/nib
MIT License
1.9k stars 250 forks source link

bunch of stuff #192

Closed notslang closed 11 years ago

notslang commented 11 years ago

see the commit messages for what's going on here

this closes:

and sets up support for Travis CI, which we need to enable. Travis CI is enabled

notslang commented 11 years ago

I've rebased commits from #94 but there is still one failing test that i'm not sure how to fix. @kizu, could you look into this? I think something in flex.styl is conflicting with the new vendor function and causing it to carry over properties from the previous rule... very odd stuff.

kizu commented 11 years ago

@slang800 I saw that those tests were added here, by @kpdecker: https://github.com/slang800/nib/commit/5c2f1d1c69c013970235950fcd6b3cf35b0d7c87, and there were a lot of commits there, so maybe he could tell what should go there? Is it looks from the tests, there should be the fix for the auto value, so it should be translated to 1 for old syntax, and some other such minor stuff.

Also, as I know, old syntax was supported only with -webkit- and -moz- prefixes, so this place could be optimized a bit more, by removing the never used prefixes from the old syntax flexbox properties.

I fear I won't have time to fix this issues by myself, but thanks anyway for getting into this issue, hope everything would resolve nicely!

notslang commented 11 years ago

ok, all 15 tests pass now!