Skip to content

docs: IE9 gradient.#17522

Merged
mdo merged 2 commits intov4-devfrom
v4-dev-xmr-ie9-gradient
Sep 19, 2015
Merged

docs: IE9 gradient.#17522
mdo merged 2 commits intov4-devfrom
v4-dev-xmr-ie9-gradient

Conversation

@XhmikosR
Copy link
Member

@XhmikosR XhmikosR commented Sep 8, 2015

before:
old

after:
new

/CC @mdo @cvrebert: the branch needs some cleanup before it's merged but I opened it for feedback.

@XhmikosR XhmikosR added this to the v4.0.0-alpha.2 milestone Sep 8, 2015
@mdo
Copy link
Member

mdo commented Sep 8, 2015

Does Autoprefixer really not give us that?

@XhmikosR
Copy link
Member Author

XhmikosR commented Sep 8, 2015

I don't see it there. I'll have a close look later again.
On Sep 8, 2015 09:36, "Mark Otto" notifications@github.com wrote:

Does Autoprefixer really not give us that?


Reply to this email directly or view it on GitHub
#17522 (comment).

@hnrch02
Copy link
Collaborator

hnrch02 commented Sep 8, 2015

@mdo Apparently not: postcss/autoprefixer#240.

@cvrebert
Copy link
Collaborator

cvrebert commented Sep 8, 2015

I guess someone could try extracting a plugin for it from cssgrace (https://github.com/cssdream/cssgrace/blob/a81be90a944b7da7c0db366ba0de1aee6098b035/lib/index.js#L303 ).

@XhmikosR
Copy link
Member Author

XhmikosR commented Sep 9, 2015

The thing is that it's only used in one place. Not sure it's worth adding a new mixin.

Other than that, I don't like my solution 100%. I'll tweak it later.

@XhmikosR XhmikosR force-pushed the v4-dev-xmr-ie9-gradient branch from 6bdf7bd to ed27fb8 Compare September 10, 2015 05:16
@XhmikosR
Copy link
Member Author

All right, I tried splitting this up.

/CC @mdo @cvrebert @hnrch02

@XhmikosR XhmikosR force-pushed the v4-dev-xmr-ie9-gradient branch from ed27fb8 to 7399607 Compare September 19, 2015 06:42
mdo added a commit that referenced this pull request Sep 19, 2015
@mdo mdo merged commit 64cdb79 into v4-dev Sep 19, 2015
@mdo mdo deleted the v4-dev-xmr-ie9-gradient branch September 19, 2015 19:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants