Bug 34302 - buttons/pictures missing at http://bscw.gmd.de
Summary: buttons/pictures missing at http://bscw.gmd.de
Status: RESOLVED FIXED
Alias: None
Product: konqueror
Classification: Applications
Component: khtml (show other bugs)
Version: 2.2.1
Platform: openSUSE Linux
: NOR normal
Target Milestone: ---
Assignee: Konqueror Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2001-10-30 22:33 UTC by hoea
Modified: 2003-01-08 11:35 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description hoea 2001-10-30 22:20:53 UTC
(*** This bug was imported into bugs.kde.org ***)

Package:           khtml
Version:           2.2.1 (using KDE 2.2.1 )
Severity:          normal
Installed from:    SuSE RPMs
Compiler:          Not Specified
OS:                Linux
OS/Compiler notes: suse 7.3

load http://bscw.gmd.de
below the banner there should be 5 buttons/pictures - konqueror shows just 3 of them

(Submitted via bugs.kde.org)
Comment 1 Daniel Naber 2001-11-27 18:11:53 UTC
Sorry but that page is broken so badly - it's not a khtml bug but a bug in the page.
Comment 2 Dirk Mueller 2003-01-08 04:17:21 UTC
not a bug in the page, but in khtml 
Comment 3 Dirk Mueller 2003-01-08 11:35:01 UTC
Subject: kdelibs/khtml/html

CVS commit by mueller: 

handle <!--> and <!--->
CCMAIL:34302-done@bugs.kde.org


  M +9 -4      htmltokenizer.cpp   1.252


--- kdelibs/khtml/html/htmltokenizer.cpp  #1.251:1.252
@@ -7,5 +7,6 @@
               (C) 1999 Lars Knoll (knoll@kde.org)
               (C) 1999 Antti Koivisto (koivisto@kde.org)
-              (C) 2001 Dirk Mueller (mueller@kde.org)
+              (C) 2001-2003 Dirk Mueller (mueller@kde.org)
+              (C) 2002 Apple Computer, Inc.
 
     This library is free software; you can redistribute it and/or
@@ -750,8 +751,12 @@ void HTMLTokenizer::parseTag(DOMStringIt
                         ++src;
                         dest = buffer; // ignore the previous part of this tag
-                        comment = true;
                         tag = NoTag;
-                        parseComment(src);
 
+                        comment = true;
+                        // push what we parsed so far upon the stack. helps for <!-->
+                        checkScriptBuffer();
+                        scriptCode[0] = scriptCode[1] = '-';
+                        scriptCodeSize = 2;
+                        parseComment(src);
                         return; // Finished parsing tag!
                     }