[prev in list] [next in list] [prev in thread] [next in thread]
List: kfm-devel
Subject: Re: khtml (fwd)
From: Koos Vriezen <koos.vriezen () xs4all ! nl>
Date: 2002-02-23 16:00:05
[Download RAW message or body]
On Sat, 23 Feb 2002, Koos Vriezen wrote:
> > If i use Konq to view the web page of the National Bank of Canada
> > (www.nbc.ca, or www.bnc.ca) I eather get konq freezing and then crashing on
> > me (kde 3.0 beta 2) or it just shows nothing (kde 2.2.2). Now, back when kde
> > 2.0 came out it could show the page just fine, but after 2.1 and newer, it
> > could no longer show the page. Now, trying to track down the exact bug, i
> > used mozilla (that can view the web page btw) to download the front page
> > onto my hard drive so i could do some testing. When i tried to view the page
> > from my harddrive with Kong, it viewed fine (besides a little wiredness like
> > duplicated banners .. but that is the fault of mozilla i think). The reason i
> > am bringing this to your attention is this. My girlfriend is the internet
> > consultant for the National Bank, and she is trying VERY hard to have kde
> > supported by the bank (and that would be toooo cool!) , But its kind of hard
> > to do when the main page for the bank will not ever show correctly.
>
> It seems there is a recursion in body.onLoad.
Besides the wrong html, window events aren't clear in a
document.open/clear call. This should fix it:
--- xml/dom_docimpl.cpp 2002/02/16 13:58:15 1.162
+++ xml/dom_docimpl.cpp 2002/02/23 15:52:02
@@ -1021,6 +1021,9 @@ void DocumentImpl::clear()
m_tokenizer = 0;
removeChildren();
+ QPtrListIterator<RegisteredEventListener> it(m_windowEventListeners);
+ for (; it.current();)
+ m_windowEventListeners.removeRef(it.current());
}
void DocumentImpl::setStyleSheet(const DOM::DOMString &url, const
DOM::DOMString &sheet)
This also renders http://www.nbc.ca/ (after I pressed reload).
>
> I've looked at the main html page and IMHO it is not correct:
>
> <SCRIPT LANGUAGE=javascript>
> function loadframe() {
> .....
> document.open();
> document.write(pag);
> document.close();
> }
> </SCRIPT>
> <BODY onload="javascript:loadframe();">
> </BODY>
>
> First, the pag string doesn't contain a <html></html> tags.
> Second, document.open overwrites the whole document including the
> javascript code. IIRC IE can have problems with this, showing a
> background wait cursor (arrow with a small hourglass).
> I think this main page should be rewritten to:
>
> function loadframe() {
> .....
> mainframe.document.open();
> mainframe.document.write(pag);
> mainframe.document.close();
> }
> </SCRIPT>
> <FRAMESET onload="javascript:loadframe();">
> <FRAME NAME='mainframe' SRC="about:blank">
> </FRAMESET>
> </HTML>
>
> I've attached the adjusted page. Please try.
>
> Koos
>
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic