Bug 117787 - Problem with window.scrollX/window.scrollY [menus in di.fm]
Summary: Problem with window.scrollX/window.scrollY [menus in di.fm]
Status: RESOLVED FIXED
Alias: None
Product: konqueror
Classification: Applications
Component: khtml ecma (show other bugs)
Version: unspecified
Platform: openSUSE Linux
: NOR normal
Target Milestone: ---
Assignee: Konqueror Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-12-06 12:58 UTC by Raul Moratalla
Modified: 2005-12-06 16:02 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
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