Bug 59454

Summary:	A <DEL> element is not always closed at the </DEL>
Product:	[Applications] konqueror	Reporter:	bert
Component:	khtml parsing	Assignee:	Konqueror Developers <konq-bugs>
Status:	RESOLVED DUPLICATE
Severity:	normal	CC:	ismail
Priority:	NOR
Version:	4.0
Target Milestone:	---
Platform:	Compiled Sources
OS:	Linux
Latest Commit:		Version Fixed In:
Attachments:	cvs-head behaviour proposed patch

Description bert 2003-06-07 00:47:26 UTC

Version:           4.0 (using KDE 3.1.2)
Installed from:    compiled sources
Compiler:          gcc version 2.95.3 20010315 (release)
OS:          Linux (i686) release 2.4.17

Konqueror/khtml doesn't close a DEL element at the corresponding </DEL> tag, if a P that is nested inside the DEL has an omitted end tag. The result is that the rest of the file is styled as deleted text. According to HTML, the following is valid and has an implied </P> just before the </DEL>, but Konqueror only works correctly if the </P> tag is inserted explicitly:

  <del><p>This is deleted.</del>
  <p>This is not.

Comment 1 Ismail Donmez 2003-06-07 09:01:47 UTC

Hi Bert, 
 
I think we got this fixed on KDE cvs-head. I am attaching a screenshot to check if its the 
behaviour you are expected. Please comment if its the right behaviour.

Comment 2 Ismail Donmez 2003-06-07 09:03:02 UTC

Created attachment 1741 [details]
cvs-head behaviour

cvs-head behaviour screenshot.

Comment 3 Ismail Donmez 2003-06-08 20:11:48 UTC

Alright I am possibly smoking crack bug is valid on KDE cvs-head too.

Comment 4 Stephan Kulow 2003-10-24 14:03:58 UTC

Created attachment 2865 [details]
proposed patch

Comment 5 bert 2004-05-11 13:26:26 UTC

Bug is still present in 3.2.2.
B.t.w., <INS> has the same behaviour, so, assuming the patch works, it will have to be duplicated for </INS> as well.

Comment 6 Stephan Kulow 2004-10-17 15:07:14 UTC

Comment on attachment 2865 [details]
proposed patch

the patch is bogus

Comment 7 Allan Sandfeld 2005-03-22 12:18:06 UTC

Can be fixed by parsing residual styles as in #20976, if you add DEL and INS to the list of residual style elements.

*** This bug has been marked as a duplicate of 20976 ***