[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-bugs-dist
Subject:    [Bug 101340] javascript form submit does not work
From:       Allan Sandfeld <kde () carewolf ! com>
Date:       2005-06-16 15:22:50
Message-ID: 20050616152250.7036.qmail () ktown ! kde ! org
[Download RAW message or body]

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
         
http://bugs.kde.org/show_bug.cgi?id=101340         
kde carewolf com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED



------- Additional Comments From kde carewolf com  2005-06-16 17:22 -------
SVN commit 426139 by carewolf:

Hack from Webcore to behave like other browsers.
BUG: 101340


 M  +22 -2     khtmlview.cpp  


--- trunk/KDE/kdelibs/khtml/khtmlview.cpp #426138:426139
 @ -3042,11 +3042,31  @
 						button,0, _mouse, dblclick );
         me->ref();
         targetNode->dispatchEvent(me,exceptioncode,true);
-        if (me->defaultHandled() || me->defaultPrevented())
+	bool defaultHandled = me->defaultHandled();
+        if (defaultHandled || me->defaultPrevented())
             swallowEvent = true;
         me->deref();
 
-        if (eventId == EventImpl::MOUSEDOWN_EVENT) {
+	// Special case: If it's a click event, we also send the KHTML_CLICK or \
KHTML_DBLCLICK event. This is not part +	// of the DOM specs, but is used for \
compatibility with the traditional onclick="" and ondblclick="" attributes, +	// as \
there is no way to tell the difference between single & double clicks using DOM (only \
the click count is +	// stored, which is not necessarily the same)
+	if (eventId == EventImpl::CLICK_EVENT) {
+	    me = new MouseEventImpl(dblclick ? EventImpl::KHTML_ECMA_DBLCLICK_EVENT : \
EventImpl::KHTML_ECMA_CLICK_EVENT, +				    \
true,cancelable,m_part->xmlDocImpl()->defaultView(), +				    \
detail,screenX,screenY,clientX,clientY,pageX,pageY, +				    \
ctrlKey,altKey,shiftKey,metaKey, +				    button,0, _mouse, dblclick);
+
+	    me->ref();
+	    if (defaultHandled)
+		me->setDefaultHandled();
+	    targetNode->dispatchEvent(me,exceptioncode,true);
+            if (me->defaultHandled() || me->defaultPrevented())
+                swallowEvent = true;
+	    me->deref();
+        }
+        else if (eventId == EventImpl::MOUSEDOWN_EVENT) {
             // Focus should be shifted on mouse down, not on a click.  -dwh
             // Blur current focus node when a link/button is clicked; this
             // is expected by some sites that rely on onChange handlers running


[prev in list] [next in list] [prev in thread] [next in thread]