Bug #56019
closedResourceCompressor: concatenateCss with forceOnTop gives wrong order
100%
Description
While testing #53116 (about concatenateJs and forceOnTop) I also checked if that part works fine for CSS. The forceOnTop works if you don't concatenate them:
page > config.concatenateCss = 0 page = PAGE page { includeCSS { b = fileadmin/b.css b { forceOnTop = 0 } c = fileadmin/c.css c { forceOnTop = 1 } a = fileadmin/a.css a { forceOnTop = 1 } } }
gives:
<link rel="stylesheet" type="text/css" href="fileadmin/a.css?1392566437" media="all">
<link rel="stylesheet" type="text/css" href="fileadmin/c.css?1392566430" media="all">
<link rel="stylesheet" type="text/css" href="typo3temp/stylesheet_993134cfc3.css?1385132976" media="all">
<link rel="stylesheet" type="text/css" href="fileadmin/b.css?1392566433" media="all">
(okay, we might look if we can get rid of that newline in the middle)
But with config.concatenateCss = 1 the order is then:- default CSS
- b
- c
- a
- a
- c
- default CSS
- b
Updated by Markus Klein almost 11 years ago
Hi! I checked that too, but forceOnTop for includeCSS is not documented to exist.
http://docs.typo3.org/typo3cms/TyposcriptReference/latest/Setup/Page/#includecss-array
Updated by Stefan Neufeind almost 11 years ago
- fix it for the concatenate-case as a "TASK" (make the functionality complete)
- document it
Updated by Gerrit Code Review almost 11 years ago
- Status changed from New to Under Review
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/27640
Updated by Gerrit Code Review almost 11 years ago
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/27640
Updated by Gerrit Code Review almost 11 years ago
Patch set 1 for branch TYPO3_6-1 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/27672
Updated by Markus Klein almost 11 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 599cdb66a8bc9ac4f72afffe704f3b6f5afc35c0.