Bug 117787

Summary:	Problem with window.scrollX/window.scrollY [menus in di.fm]
Product:	[Applications] konqueror	Reporter:	Raul Moratalla <raul.moratalla>
Component:	khtml ecma	Assignee:	Konqueror Developers <konq-bugs>
Status:	RESOLVED FIXED
Severity:	normal	CC:	maksim
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	openSUSE
OS:	Linux
Latest Commit:		Version Fixed In:

Description Raul Moratalla 2005-12-06 12:58:33 UTC

Version:            (using KDE KDE 3.5.0)
Installed from:    SuSE RPMs
OS:                Linux

When you select a submenu from the menu in www.di.fm you can see displayed the submenu but always in the same position, top of menu.

Comment 1 Maksim Orlovich 2005-12-06 15:33:01 UTC

For some reason scrollX/scrollY returned Undefined here, causing the problem. Very odd.

Comment 2 Maksim Orlovich 2005-12-06 15:57:03 UTC

SVN commit 486021 by orlovich:

These work a lot better if one actually binds them.
Fixes submenu positionning on di.fm. 
(Yey for menu packages that use non-obfuscated, well-commented code!)
BUG:117787


 M  +2 -0      kjs_window.cpp  


--- branches/KDE/3.5/kdelibs/khtml/ecma/kjs_window.cpp #486020:486021
@@ -260,6 +260,8 @@
   scroll	Window::Scroll		DontDelete|Function 2
   scrollBy	Window::ScrollBy	DontDelete|Function 2
   scrollTo	Window::ScrollTo	DontDelete|Function 2
+  scrollX       Window::ScrollX         DontDelete|ReadOnly
+  scrollY       Window::ScrollY         DontDelete|ReadOnly
   moveBy	Window::MoveBy		DontDelete|Function 2
   moveTo	Window::MoveTo		DontDelete|Function 2
   resizeBy	Window::ResizeBy	DontDelete|Function 2

Comment 3 Maksim Orlovich 2005-12-06 16:02:11 UTC

SVN commit 486024 by orlovich:

Minimal test for scrollX/scrollY being thee
CCBUG:117787


 M             window.js