Project

General

Profile

Actions

Bug #32163

closed

@charset must be lowercase in CSS

Added by Jan Schreier almost 13 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Form Framework
Target version:
-
Start date:
2011-11-29
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
4.6
PHP Version:
Tags:
Complexity:
no-brainer
Is Regression:
Sprint Focus:

Description

In file

sysext/Form/Resources/Public/CSS/Form.css

the first line is
@CHARSET "UTF-8";
but it must be:
@charset "UTF-8";

rule: http://www.w3.org/TR/CSS21/syndata.html#charset

I noticed the problem because scriptmerger does look for @charset in lowercase only when merging css

Actions #1

Updated by Mr. Jenkins almost 13 years ago

  • Status changed from New to Under Review

Patch set 1 of change I71436dfd4a74da8ee5592e6755565d96d160d1fe has been pushed to the review server.
It is available at http://review.typo3.org/6979

Actions #2

Updated by Markus Klein almost 13 years ago

Thanks for reporting this Jan!
Good catch.

Also found an instance in the stylesheet for the Installer.

Actions #3

Updated by Markus Klein almost 13 years ago

Note for the backports: Forms are only available 4.6+, so this has to be removed for the lower versions.

Actions #4

Updated by Gerrit Code Review almost 13 years ago

Patch set 1 for branch TYPO3_4-6 has been pushed to the review server.
It is available at http://review.typo3.org/7348

Actions #5

Updated by Gerrit Code Review almost 13 years ago

Patch set 1 for branch TYPO3_4-5 has been pushed to the review server.
It is available at http://review.typo3.org/7350

Actions #6

Updated by Markus Klein almost 13 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #7

Updated by Gerrit Code Review almost 13 years ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch TYPO3_4-4 has been pushed to the review server.
It is available at http://review.typo3.org/7351

Actions #8

Updated by Markus Klein almost 13 years ago

  • Status changed from Under Review to Resolved
Actions #9

Updated by Benni Mack about 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF