rlamana / prefix.less

No more complex and fancy LESS mixins, just CSS3. Make your code standard and prefix-less.
7 stars 1 forks source link

flex-flow ignoring the direction part in 2009 specs #1

Open katerlouis opened 9 years ago

katerlouis commented 9 years ago

I had an issue using your .flex-flow mixin. It worked well for the tweener and new specs. But when I use it with 2 parameters, which is basically the whole thing behind flex-flow as I understand it, the "when"-statement of the mixin is not fullfilled so the 2009 specs aren't in the output CSS.

So I had to use .flex-direction and .flex-wrap seperately, which is no biggy- but figuring that out was one!

If I'm wrong, please correct me here. If not, then maybe you should consider editing the mixin in a two variable mixin or not support it and point out clearly to use .flex-direction and .flex-wrap instead!

Because this prefix-thing rocks!

Thx so far! :)

Kater!

rlamana commented 9 years ago

Hi @katerlouis! Thanks for the feedback, LESS mixins are a bit limited and I also found it hard to make it work with 2 parameters.

However I'm not updating this project anymore. I suggest you to use an automatic prefixer like https://github.com/postcss/autoprefixer. It also have grunt and gulp plugins to automate the generation of your CSS and forget about prefixes forever :-D

Cheers!

katerlouis commented 9 years ago

I just figured out a horrible thing .. - It seem's im right, and I "fixed" your mixin. But.. the effort was for nothing! Almost every Browser that only supports 2009 specs also doesnt support wrapping, COMPLETELY ...