Bug 119327 - Sliders on a web page does not work
Summary: Sliders on a web page does not work
Alias: None
Product: konqueror
Classification: Applications
Component: khtml (show other bugs)
Version: unspecified
Platform: openSUSE Linux
: NOR normal
Target Milestone: ---
Assignee: Maksim Orlovich
Depends on:
Reported: 2006-01-01 00:48 UTC by Osho GG
Modified: 2006-01-22 02:26 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:

patch (503 bytes, patch)
2006-01-22 01:37 UTC, Maksim Orlovich

Note You need to log in before you can comment on or make changes to this bug.
Description Osho GG 2006-01-01 00:48:33 UTC
Version:            (using KDE KDE 3.5.0)
Installed from:    SuSE RPMs
OS:                Linux


I have noticed that some sliders do not work on this webpage. The webpage is


If you look at the left, there is a Search Restaurants bar. Under that there are various sections titled "By Cuisine" , "By Feature" or "By Price". There are little sliders next to it. I am unable to use these sliders to slide up/down. Firefox/IE works fine on this page.

Comment 1 Maksim Orlovich 2006-01-22 01:33:52 UTC
OK, have a fix, and it has nothing to do with drags or mouse events or whatsoever :-). We just errored out for an unrelated reason, breaking JS here.
Comment 2 Maksim Orlovich 2006-01-22 01:37:15 UTC
Created attachment 14333 [details]

This adds "poor man's" support for getComputedStyle on hidden nodes. It doesn't
actually return anything but "undefined" but this is enough to get the sliders
working. OTOH, there is still a weird bug where all movement starts selecting
Comment 3 Maksim Orlovich 2006-01-22 02:26:29 UTC
SVN commit 501090 by orlovich:

Go ahead and return an object when doing getComputedStyle on renderer-less elements.
It wouldn't return anything other than "undefined" in response to any queries,
but that's enough for some websites, apparently.


 M  +1 -1      dom2_viewsimpl.cpp  

--- branches/KDE/3.5/kdelibs/khtml/xml/dom2_viewsimpl.cpp #501089:501090
@@ -41,7 +41,7 @@
 CSSStyleDeclarationImpl *AbstractViewImpl::getComputedStyle(ElementImpl* elt, DOMStringImpl* /*pseudoElt*/)
-    if (!elt || !elt->renderer())
+    if (!elt)
         return 0;
     CSSStyleDeclarationImpl* style = new RenderStyleDeclarationImpl( elt );