Skip to content
This repository has been archived by the owner on Jun 6, 2022. It is now read-only.

Allow custom media to reference each other #10

Merged
merged 1 commit into from
Jun 30, 2015
Merged

Allow custom media to reference each other #10

merged 1 commit into from
Jun 30, 2015

Conversation

hgl
Copy link
Contributor

@hgl hgl commented Apr 20, 2015

The final missing piece. :)

Any plan on settling the option name? I plan to submit the extract pr once it's resolved.

@MoOx
Copy link
Contributor

MoOx commented Apr 25, 2015

looks good.
I will go for "computed-definitions" for the name.

@@ -1,3 +1,4 @@
@custom-media --a (--a);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this fixture seems not used.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's used by the "should transform custom media with self reference" test.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry I don't see how. No import or something in test/fixtures/transform-self-reference.css

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, you are right. It's been awhile and I have no idea why this file exists. Let me double check.

@MoOx
Copy link
Contributor

MoOx commented Jun 30, 2015

Please remove the unused fixtures and rebase on master so I can merge.

@hgl
Copy link
Contributor Author

hgl commented Jun 30, 2015

PTAL

@MoOx
Copy link
Contributor

MoOx commented Jun 30, 2015

Nice.

MoOx added a commit that referenced this pull request Jun 30, 2015
Allow custom media to reference each other
@MoOx MoOx merged commit 590e414 into csstools:master Jun 30, 2015
@MoOx
Copy link
Contributor

MoOx commented Jun 30, 2015

Released as 4.1.0

@hgl
Copy link
Contributor Author

hgl commented Jun 30, 2015

Thanks. I will send the cssnext PR soon.

@tbck
Copy link
Contributor

tbck commented Jan 30, 2016

This PR fixes #9.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants