#6634 closed bug (fixed)
minified css does not @include minified css files
Reported by: | WanderingZombie | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | 1.8.7 |
Component: | [meta] ui.build | Version: | 1.8.6 |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
themes/base/minified/jquery.ui.all.min.css
@import "jquery.ui.base.css";@import "jquery.ui.theme.css";
Clearly this is not what the user intended when referencing a minified file. It should be :
@import "jquery.ui.base.min.css";@import "jquery.ui.theme.min.css";
OR, as minified files are in a directory named minified, there's no real need to change the filename of each .css file.
Change History (5)
comment:1 Changed 13 years ago by
Milestone: | TBD → 1.9 |
---|
comment:2 Changed 13 years ago by
Component: | ui.core → [meta] ui.build |
---|
comment:3 Changed 13 years ago by
This does the trick, put into minify target:
<replaceregexp match=".css" replace=".min.css" flags="g"> <fileset dir="${dist.dir}/themes/base/minified/"> <include name="*.base.min.css"/> <include name="*.all.min.css"/> </fileset> </replaceregexp>
comment:4 Changed 13 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
Build: Replace filename references in base/all.min.css. Fixes #6634 - minified css does not @include minified css files.
Also make a copy of images to fix relative paths.
Changeset: 812c274a3376c4fee4d9705628a629d185bc86a9
comment:5 Changed 12 years ago by
Milestone: | 1.9 → 1.8.7 |
---|
Note: See
TracTickets for help on using
tickets.
Good catch. Missed that when adding minified CSS files.