On 01.08.2011 13:22, Albrecht Schlosser wrote: > On 01.08.2011 12:35, MacArthur, Ian (SELEX GALILEO, UK) wrote: > >>> Could you please test with svn releases pre-8866 (STR #2680) and >>> if that doesn't solve it with pre-8639 (STR #2613)? I'm short of >>> time, otherwise I'd do it. Thanks. >> >> Testing: >> >> r8860 - broken >> >> r8630 - OK > > Thanks. > >> So I guess that means it broke somewhere between 8630 and 8860, and that >> perhaps the fix to #2613 might be in the frame here? > > Yup, that's what I suspected :-(
... and that was wrong! The culprit is in svn rev. 8724 (STR #2640): "Fl::w() and Fl::h() are documented to return the width and height of the main screen's work area (system-occupied space excluded); Fl::screen_xywh() functions instead compute the full width and height of a screen." So it looks as if the menu scrolling code looks at the wrong screen height value. I didn't investigate this further yet, but this can be seen with a simple test: I used test/menubar to test, and I found that it worked in my Linux VM, but it didn't work on Windows. When I tested with the old (fltk 1.1 or 1.3/r 8630) version(s), I saw that the scrolling was done when (or before) the cursor entered the Windows task bar. I moved the window to my 2nd screen (w/o task bar), and there the current version worked as expected. Another test with the Windows task bar set to "hide automatically" (translated from German Windows version) showed that it worked as well (now at the screen border, as expected). The problem seems to be that we don't get mouse events as soon as the cursor enters the Windows task bar, and thus we may need to change the menu code to account for this (and the changed behavior documented in the fix for STR #2640). Thoughts, anybody? Note: I won't have the time to dig into the menu code for the next days or a week, so if anybody could take a look, that would be fine. Albrecht _______________________________________________ fltk-dev mailing list [email protected] http://lists.easysw.com/mailman/listinfo/fltk-dev
