New Tokens for Concatenation #95

Closed
roblarsen opened this Issue Aug 8, 2012 · 5 comments

Comments

Projects
None yet
2 participants
Owner

roblarsen commented Aug 8, 2012

Since we're no longer educating in the main project with the comments, we can move to something that's more obviously designed as a hook for the build process. Part of me just wants to use something obvious:

<!--@concat@-->
<!--@@concat@@->

The rest of me wants ascii art:

<!--\o/_\o/-->
<!--\o/_\o/_\o/->
Owner

roblarsen commented Sep 1, 2012

Anyone interested in how this is going to work should look at

https://github.com/h5bp/ant-build-script/tree/new-token

In addition to a new token, one that isn't quite as succinct as I would like, we're going to have the ability to wrap any set of tags in comments and pull out an attribute from them

right now all it supports is:

<!-- //-beg- concat_js -->
<script src="js/libs/jquery-1.8.0.js"></script>
<script src="js/libs/jquery-ui-1.8.20.custom.min.js"></script>
<!-- //-end- concat_js -->

but... we'll be able to do something like:

<!-- //-beg- concat_css -->
<link rel="stylesheet" href="css/normalize.css">
<link rel="stylesheet" href="css/main.css">
<!-- //-end- concat_css -->

and automatically concatenate CSS files in the same way we're doing the JS right now.

@roblarsen roblarsen closed this Jan 15, 2013

Anything new here?

Owner

roblarsen commented Feb 12, 2013

Yes, as you can see the issue is closed. The documentation is here: https://github.com/h5bp/ant-build-script/wiki/How-to-bypass-the-automatic-script-concatenation

Sorry, my question is about css concatenation, i cant find documentation for that.
By default i got 2 css files:
normalize.css (any easy publish deactivation possible (i can not reference in site head, but ant build still copy to publish dir), for people like me using another normalize)
main.css -> at the moment i @import my custom css here, but your example above suggests that i could be possible wihtout importing, but directly reference more css like the main.css in the head element.

Thanks a lot

Owner

roblarsen commented Feb 13, 2013

Yes I'm tracking that in #138

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment