Project

General

Profile

Actions
Copy link

Feature #18518

closed

css parsing issue: classes with element context selectors are not honoured inside other context selectors

Added by Uschi Renziehausen over 16 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Should have
Assignee:
Stanislas Rolland
Category:
-
Target version:
-
Start date:
2008-03-30
Due date:
% Done:

0%

Estimated time:
PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

Hi Stan,

the css parser has a nasty little flaw. If I declare a tag-class with a context selector in front of it, the class is not recognized. Example:

.tb-zebra-rows tr.odd {foo:bar}

That means I need to define the class extra without a context selector again without any value:

tr.odd {}

Prosit, Uschi

(issue imported from #M7963)

Files

Download all files
bug_0007963.diff (3.08 KB) bug_0007963.diff Administrator Admin, 2008-11-12 16:07
patch_0007963.diff (4.1 KB) patch_0007963.diff Administrator Admin, 2008-11-29 22:07
Actions
Copy link
 #1

Updated by Mikkel Ricky about 16 years ago

Patch for resolving this bug will be uploaded shortly.

The patch fixes "BlockStyle.parseSelectorText" such that ALL classes in a selector rule will be handled. The selector

.content table.fullwidth thead th.number

will result in three classes being available: "content" (for all elements), "fullwidth" (for table), and "number" (for th).

Actions
Copy link
 #2

Updated by Mikkel Ricky almost 16 years ago

The previously uploaded patch missed fixing the parsing problem in "TextStyle.parseSelectorText", and a new patch will be uploaded shortly.

Actions
Copy link
 #3

Updated by Stanislas Rolland almost 16 years ago

Thanks for the patch Mikkel. I will test it. I was indeed going to comment that something similar would be needed for TextStyle.

I am afraid Uschi may not have time to test this.

Actions
Copy link
 #4

Updated by Stanislas Rolland almost 16 years ago

Committed to SVN TYPO3core trunk (revision 4557).

Actions
Copy link
 #5

Updated by Benni Mack about 6 years ago

  • Status changed from Resolved to Closed
Actions
Copy link

Also available in: Atom PDF