Opened 13 years ago

Closed 13 years ago

Last modified 12 years ago

#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 Jörn Zaefferer

Milestone: TBD1.9

Good catch. Missed that when adding minified CSS files.

comment:2 Changed 13 years ago by Jörn Zaefferer

Component: ui.core[meta] ui.build

comment:3 Changed 13 years ago by Jörn Zaefferer

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 Jörn Zaefferer

Resolution: fixed
Status: newclosed

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 Scott González

Milestone: 1.91.8.7
Note: See TracTickets for help on using tickets.