Bug #69648

Built-in css minification destroys certain CSS constructs

Added by Jost Baron almost 4 years ago. Updated almost 4 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2015-09-10
Due date:
% Done:

0%

TYPO3 Version:
6.2
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

Example:

top: calc(100% + 5px);

is compressed to

top:calc(100%+5px);

The latter on is invalid and thus ignored by browsers. From here :

Note: The + and - operators must always be surrounded by whitespace.
The operand of calc(50% -8px) for instance will be parsed as a percentage
followed by a negative length, an invalid expression, while the operand
of calc(50% - 8px) is a percentage followed by a minus sign and a length.
Even further, calc(8px + -50%) is treated as a length followed by a plus
sign and a negative percentage. The * and / operators do not require
whitespace, but adding it for consistency is allowed, and recommended.


Related issues

Related to TYPO3 Core - Bug #62463: bug in compressing the css-files!? Closed 2014-10-25
Related to TYPO3 Core - Bug #64950: Regex in RessourceCompressor triggers PHP segfault with old pcre versions Closed 2015-02-09
Duplicates TYPO3 Core - Bug #69309: config.compressCss breaks CSS3 calc() value Closed 2015-08-25

History

#1 Updated by Christian Kuhn almost 4 years ago

Hey Jost. We've fiddled with this stuff already and it is fixed in 7. in 6.2 however it may lead to pcre segfaults and this is much worse than the broken compression. Probably, we will not make additional attempts to still fix this issue in 6.2 anymore.

#2 Updated by Daniel Goerz almost 4 years ago

  • Status changed from New to Closed

Closed as duplication of #69309 and #62463

Also available in: Atom PDF