[10:03:56 CDT(-0500)] <EricDalquist> so sass-maven-plugin is done athena
[10:04:09 CDT(-0500)] <athena> yay!!
[10:04:13 CDT(-0500)] <athena> that's awesome 
[10:04:18 CDT(-0500)] <EricDalquist> posted some questions to the list
[10:04:20 CDT(-0500)] <athena> it working well w/ our current setup?
[10:04:21 CDT(-0500)] <athena> ah ok
[10:04:27 CDT(-0500)] <EricDalquist> the biggest being … do we delete the .css files from git
[10:04:35 CDT(-0500)] <EricDalquist> and just generate them at build time
[10:04:44 CDT(-0500)] <EricDalquist> but if you could take a look at that email and let me know
[10:05:04 CDT(-0500)] <athena> yeah
[10:05:06 CDT(-0500)] <athena> just glanced over it
[10:05:23 CDT(-0500)] <athena> my suggestion is we delete the css from git and document really well how to change it back, ig uess
[10:05:27 CDT(-0500)] <athena> but want to see community feedback
[10:05:36 CDT(-0500)] <EricDalquist> if we do build-time generation I'm guessing we'll need to do a little work on the resource server plugin
[10:05:48 CDT(-0500)] <EricDalquist> since that looks for the css in the source dir
[10:06:16 CDT(-0500)] <athena> can you try running the script w/ the "expanded" option?
[10:06:19 CDT(-0500)] <athena> that might be part of the formatting
[10:06:26 CDT(-0500)] <athena> that makes sense
[10:06:34 CDT(-0500)] <EricDalquist> sure
[10:10:39 CDT(-0500)] <EricDalquist> having all this scripting lanaguages available via java is handy though
[10:15:05 CDT(-0500)] <EricDalquist> that was it
[10:15:16 CDT(-0500)] <EricDalquist> changed the style to expanded and it works
[10:16:55 CDT(-0500)] <athena> oh great!
[10:16:55 CDT(-0500)] <athena> ok
[10:16:57 CDT(-0500)] <EricDalquist> so if we ditch the .css files in git
[10:17:14 CDT(-0500)] <EricDalquist> then I think we bind the sass maven plugin to generate-resources
[10:17:28 CDT(-0500)] <athena> then we can look for the stuff in target?
[10:17:33 CDT(-0500)] <EricDalquist> yeah
[10:17:42 CDT(-0500)] <EricDalquist> I'll have to look at the resource server plugin
[10:17:52 CDT(-0500)] <EricDalquist> and see if it is configurable enough as is
[10:17:55 CDT(-0500)] <EricDalquist> or if we need more work
[10:18:10 CDT(-0500)] <EricDalquist> I'm looking at http://maven.apache.org/guides/introduction/introduction-to-the-lifecycle.html#Lifecycle_Reference
[10:18:32 CDT(-0500)] <EricDalquist> so it looks like we actually bind the resource-server-plugin to prepare-package
[10:19:18 CDT(-0500)] <EricDalquist> I'm guessing I did that so that we didn't have to wait for the skin compile every time we built
[10:19:27 CDT(-0500)] <EricDalquist> so maybe we bind this SCSS->CSS to prepare-package too
[10:19:36 CDT(-0500)] <EricDalquist> and just have it listed before resource-server so it runs first
[10:22:40 CDT(-0500)] <athena> makes sense to me
[10:22:56 CDT(-0500)] <EricDalquist> I'll see if I can make that work on the branch
[10:23:07 CDT(-0500)] <athena> we'll also want to document how to get sass to run and copy files to the deployed portal during development
[10:23:13 CDT(-0500)] <athena> think peter should be able to help w/ that
[10:23:14 CDT(-0500)] <EricDalquist> the hardest part of that plugin was that I've never run sass before 
[10:23:19 CDT(-0500)] <athena> yeah
[10:23:34 CDT(-0500)] <EricDalquist> peter's pull on friday really helped though
[10:25:03 CDT(-0500)] <athena> oh that's great
[10:25:09 CDT(-0500)] <athena> and i'm so glad you were able to write the plugin
[10:25:19 CDT(-0500)] <athena> i've actually been sort of worried about this in the back of my mind for a while
[10:25:47 CDT(-0500)] <EricDalquist> https://github.com/Jasig/sass-maven-plugin/blob/master/src/main/java/org/jasig/maven/plugin/sass/SassCompilerMojo.java
[10:25:51 CDT(-0500)] <EricDalquist> that is the entire thing
[10:26:56 CDT(-0500)] <EricDalquist> yeah looks like resource server plugin needs some TLC to get this working
[10:38:38 CDT(-0500)] <athena> ah
[13:47:42 CDT(-0500)] <EricDalquist> peterjhart: thanks for the SASS cleanup
[13:47:47 CDT(-0500)] <EricDalquist> made getting the maven plugin working much easier
[14:14:39 CDT(-0500)] <peterjhart> sweet
[14:15:09 CDT(-0500)] <peterjhart> is the maven plugin complete?
[14:16:32 CDT(-0500)] <EricDalquist> yes
[14:16:34 CDT(-0500)] <EricDalquist> brb
[14:20:21 CDT(-0500)] <EricDalquist> well it is functional
[14:20:25 CDT(-0500)] <EricDalquist> I need to cut a release of it
[14:20:35 CDT(-0500)] <EricDalquist> but I was going to wait on that until I figure out some other changes
[14:20:45 CDT(-0500)] <EricDalquist> we have to tweak the aggregator plugin
[14:20:52 CDT(-0500)] <EricDalquist> to play nice with the sass plugin
[14:23:26 CDT(-0500)] <peterjhart> cool
[14:23:27 CDT(-0500)] <peterjhart> in sass we have been using the "expanded" style
[14:23:32 CDT(-0500)] <EricDalquist> yup
[14:23:33 CDT(-0500)] <peterjhart> setting
[14:23:39 CDT(-0500)] <EricDalquist> jen piinted that out to me
[14:23:46 CDT(-0500)] <EricDalquist> and I fixed it as the default in the plugin
[14:23:54 CDT(-0500)] <peterjhart> k
[15:38:20 CDT(-0500)] <EricDalquist> oi … we are seriously overloading the number of things being done during prepare-package
[15:40:06 CDT(-0500)] <EricDalquist> also it looks like the resource aggr plugin is good as is
[15:40:17 CDT(-0500)] <EricDalquist> I think I have build-time generation of CSS from SASS working
[16:11:23 CDT(-0500)] <EricDalquist> athena: you there?
[16:12:16 CDT(-0500)] <athena> yep
[16:12:38 CDT(-0500)] <EricDalquist> I just noticed when testing this skin scss/aggr stuff that there are a few .css files geting loaded via @Import
[16:12:54 CDT(-0500)] <EricDalquist> that are not minified or have good caching headers
[16:12:58 CDT(-0500)] <athena> ick!
[16:13:04 CDT(-0500)] <athena> that sounds like somethign that should get fixed
[16:13:06 CDT(-0500)] <EricDalquist> fss-framework-1.4.0.css has a few
[16:13:07 CDT(-0500)] <athena> which files are they?
[16:13:12 CDT(-0500)] <EricDalquist> @import url('fss-reset-global.css');@import url('fss-base-global.css');
[16:13:13 CDT(-0500)] <athena> oh, hmm.
[16:13:26 CDT(-0500)] <athena> we probably don't even have those files