Opened 14 years ago
Closed 14 years ago
#1895 closed Bug (Fixed)
Bugs in _GUIScrollBars_Init()
| Reported by: | neil.wilson@… | Owned by: | Jon |
|---|---|---|---|
| Milestone: | 3.3.7.2 | Component: | Standard UDFs |
| Version: | 3.3.6.1 | Severity: | None |
| Keywords: | GUIScrollBars_Init | Cc: |
Description ¶
In _GUIScrollBars_Init(), when setting up the $tSCROLLINFO struct for both horizontal and vertical scroll bars, the code in _GUIScrollBars_Init() assigns the fMask as follows:
DllStructSetData($tSCROLLINFO, "fMask", BitOR($_SCROLLBARCONSTANTS_SIF_PAGE, $_SCROLLBARCONSTANTS_SIF_PAGE))
This is incorrect as the range and page are being set for both scrollbars and should be:
DllStructSetData($tSCROLLINFO, "fMask", BitOR($_SCROLLBARCONSTANTS_SIF_RANGE, $_SCROLLBARCONSTANTS_SIF_PAGE))
Also _GUIScrollBars_Init() states that -1 can be given for the $iHMax and the $ivMax. If however, $ivMax is not given or $ivMax = -1, then nMax in the $tSCROLLINFO for the vertical scrollbar will = -1 and not the default value of 27.
DllStructSetData($tSCROLLINFO, "nMax", $ivMax)
This should be:
DllStructSetData($tSCROLLINFO, "nMax", $aSB_WindowInfo[$index][7])
Change History (2)
comment:1 Changed 14 years ago by Jon
- Component changed from AutoIt to Standard UDFs
- Owner set to Gary
comment:2 Changed 14 years ago by Jon
- Milestone set to 3.3.7.2
- Owner changed from Gary to Jon
- Resolution set to Fixed
- Status changed from new to closed
Guidelines for posting comments:
- You cannot re-open a ticket but you may still leave a comment if you have additional information to add.
- In-depth discussions should take place on the forum.
For more information see the full version of the ticket guidelines here.
Fixed by revision [6000] in version: 3.3.7.2