Project

General

Profile

Actions

Bug #19387

closed

htmlArea RTE: Incorrect assigment of class to one line <ol> in Firefox

Added by Joachim Schmidt almost 16 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Should have
Category:
-
Target version:
-
Start date:
2008-09-29
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

I use ordered list with other counters (abc, ABC, greek). With multilines all works well, but with a single line, the class attribute ends in the <li> tag, not the <ol> tag as needed.
In my two 4.2.1 Installations (1x Windows, 1x Linux) the error is reproduceable.

(issue imported from #M9444)


Files

Fehler.jpg (103 KB) Fehler.jpg Administrator Admin, 2008-09-29 10:10
rtehtmlarea_bugfix_9444_trunk.patch (839 Bytes) rtehtmlarea_bugfix_9444_trunk.patch Administrator Admin, 2008-10-02 16:33
rtehtmlarea_bugfix_9444_typo3_4-2.patch (839 Bytes) rtehtmlarea_bugfix_9444_typo3_4-2.patch Administrator Admin, 2008-10-02 16:33
Actions #1

Updated by Stanislas Rolland almost 16 years ago

In what browser(s) do you get this error?

Actions #2

Updated by Stanislas Rolland almost 16 years ago

I did some test with IE7 and FF3.

I could not produce any problem with IE7.

Note that when there is only one item in the list, the only way to select the list itself (ol) rather then the list item (li) is to use the status bar at the bottom of the editing area, and click on the ol element. Selecting text with the cursor can only select the list item (li).

In FF3 however, I noted that when there is only one item in the list, I cannot assign a class to the list (ol) or remove a class from it even if I select it in the status bar at the bottom of the editing area.

I order to avoid assigning a class to the wrong type of element, you should remove it from the dropdown for that type. You can do that by qualifying the selector in your style sheet (ol.classname { } but not li.classname { } and not .classname { }). You may also configure the list of allowed classes for the ol tag and for the li tag in the blockstyle button configuration.

Actions #3

Updated by Joachim Schmidt almost 16 years ago

In what browser(s) do you get this error?

it is FF2, FF3 has some features I don't like!
You are right, IE7 has no problem.

I use
ol.liste-abc { list-style-type: lower-alpha;}
ol.liste-griechisch { list-style-type: lower-greek;}
ol.liste-gross-abc { list-style-type: upper-alpha;}
--> there should be no class possible on li tag

I select it in the status bar at the bottom of the editing area

I always do it this way in my tests.

In FF3 however, I noted that when there is only one item in the list, I cannot assign a class to the list (ol) or remove a class from it

I think, that is my problem. With multiple items there is no problem! In the jpg you can see, that the class is passed to the li tag.

Actions #4

Updated by Stanislas Rolland over 15 years ago

I did some further tests with Safari, Chrome and Opera. This is a Firefox-specific issue.

Actions #5

Updated by Stanislas Rolland over 15 years ago

The attached patch fixes the issue.

Actions #6

Updated by Stanislas Rolland over 15 years ago

Committed to SVN TYPO3core branch TYPO3_4-2 (revision 4272) (for inclusion in TYPO3 4.2.2) and trunk (revision 4273).

Actions #7

Updated by Joachim Schmidt over 15 years ago

I made the patch, cleared all possible caches, but -- the problem is still there in FF2.

Actions #8

Updated by Stanislas Rolland over 15 years ago

After applying the patch, you need to delete all files in directory typo3temp/rtehtmlarea and clear the browser cache.

Actions #9

Updated by Joachim Schmidt over 15 years ago

you need to delete all files in directory typo3temp/rtehtmlarea

That was it! I didn't know about that cache.

Thank you for this fast solution.

Actions #10

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF