Bug 163014

Summary:	Broken numbering in nested ol elements (invalid HTML)
Product:	[Applications] konqueror	Reporter:	Brian Sherson <caretaker82>
Component:	khtml	Assignee:	Konqueror Bugs <konqueror-bugs-null>
Status:	RESOLVED UNMAINTAINED
Severity:	minor	CC:	andresbajotierra, finex, flgalera, frank78ac
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	openSUSE
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:
Attachments:	attachment-25379-0.html

Description Brian Sherson 2008-06-01 19:42:01 UTC

Version:            (using KDE 3.5.9)
Installed from:    SuSE RPMs
OS:                Linux

In a page with nested ol elements, the numbering of the top level is affected by the second level. For instance, the following code:

<ol>
	<li>Top level item 1</li>
	<ol>
		<li>Second level 1.1</li>
		<li>Second level 1.2</li>
		<li>Second level 1.3</li>
	</ol>
	<li>Top level item 2</li>
	<ol>
		<li>Second level 2.1</li>
		<li>Second level 2.2</li>
	</ol>
	<li>Top level item 3</li>
	<ol>
		<li>Second level 3.1</li>
		<li>Second level 3.2</li>
		<li>Second level 3.3</li>
	</ol>
</ol>

Produces the following in KHTML:

1. Top level item 1
   1. Second level 1.1
   2. Second level 1.2
   3. Second level 1.3
4. Top level item 2
   1. Second level 2.1
   2. Second level 2.2
3. Top level item 3
   1. Second level 3.1
   2. Second level 3.2
   3. Second level 3.3

It appears that the top level and second level are on the same counter, and as such, everytime a second level begins, that counter is reset.

Comment 1 FiNeX 2008-06-02 01:39:02 UTC

Confirmed in konqueror 4 from trunk too.

Comment 2 Dario Andres 2008-12-27 01:21:25 UTC

Still valid in current trunk. Here:
Qt: 4.4.3 + qt-copy-patches-889120
KDE: 4.1.86 (KDE 4.1.86 (KDE 4.2 >= 20081221))
kdelibs svn rev. 901624 / kdebase svn rev. 901624
on ArchLinux x86_64 - Kernel 2.6.27.10

Comment 3 Frank Reininghaus 2009-05-25 22:51:13 UTC

Note that the test case is invalid HTML: everything inside the top level <ol>...</ol> must be inside one of the <li>...</li> environments. With the following modified version, the numbering is OK for me in current trunk:

<ol>
        <li>Top level item 1
        <ol>
                <li>Second level 1.1</li>
                <li>Second level 1.2</li>
                <li>Second level 1.3</li>
        </ol></li>
        <li>Top level item 2
        <ol>
                <li>Second level 2.1</li>
                <li>Second level 2.2</li>
        </ol></li>
        <li>Top level item 3
        <ol>
                <li>Second level 3.1</li>
                <li>Second level 3.2</li>
                <li>Second level 3.3</li>
        </ol></li>
</ol>

Comment 4 Dario Andres 2009-09-06 18:09:51 UTC

Here using:

Qt: 4.5.2 (KDE-Qt git commit 5b7a2eb42acfdea07c6075556cb43e2c95852145
        Date:   Tue Jul 28 14:10:47 2009 -0300)
KDE: 4.3.67 (KDE 4.3.67 (KDE 4.4 >= 20090904))
kdelibs svn rev. 1020545 / kdebase svn rev. 1020548
on ArchLinux i686 - Kernel 2.6.30.5

The valid HTML is shown properly. The invalid one isn't (Firefox3.5 shows it "properly"). Updating title and changing priority.

Comment 5 FiNeX 2010-09-19 15:49:07 UTC

Could bug #132590 be related to this one?

Comment 6 FiNeX 2010-09-19 17:41:30 UTC

*** Bug 145037 has been marked as a duplicate of this bug. ***

Comment 7 Justin Zobel 2021-03-21 00:25:17 UTC

Thank you for the bug report.

As this report hasn't seen any changes in 10 years or more, we ask if you can please confirm that the issue still persists.

If this bug is no longer persisting or relevant please change the status to resolved.

Comment 8 Francisco Galera 2021-03-29 14:12:10 UTC

Created attachment 137158 [details]
attachment-25379-0.html

 I do not remember this bug. I changed to i3wm some years ago. I do not use kde nowadays.Thankyou, anyway.Best,FG

    En domingo, 21 de marzo de 2021 1:25:21 CET, Justin Zobel <bugzilla_noreply@kde.org> escribió:  
 
 https://bugs.kde.org/show_bug.cgi?id=163014

--- Comment #7 from Justin Zobel <justin.zobel@gmail.com> ---
Thank you for the bug report.

As this report hasn't seen any changes in 10 years or more, we ask if you can
please confirm that the issue still persists.

If this bug is no longer persisting or relevant please change the status to
resolved.

Comment 9 Christoph Cullmann 2024-05-06 18:38:27 UTC

Dear user,

KHTML (and KJS) was a long time more or less unmaintained and got removed in KF6.

Please migrate to use a QWebEngine based HTML component.

We will do no further fixes or improvements to the KF5 branches of these components beside important security fixes.

For security issues, please see:

https://kde.org/info/security/

Sorry that we did not fix this issue during the life-time of KHTML.

Greetings
Christoph Cullmann