Bug 59454 - A <DEL> element is not always closed at the </DEL>
Summary: A <DEL> element is not always closed at the </DEL>
Status: RESOLVED DUPLICATE of bug 20976
Alias: None
Product: konqueror
Classification: Applications
Component: khtml parsing (show other bugs)
Version: 4.0
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Konqueror Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-06-07 00:47 UTC by bert
Modified: 2005-03-22 12:18 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:

Attachments
cvs-head behaviour (81.31 KB, image/png)
2003-06-07 09:03 UTC, Ismail Donmez 		Details
proposed patch (827 bytes, patch)
2003-10-24 14:03 UTC, Stephan Kulow 		Details
Show Obsolete (2) View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
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 ***