From 170225c626f3e304a4bbfd1b1bb0ec9b3c52968f Mon Sep 17 00:00:00 2001 From: Pietro Gagliardi Date: Tue, 29 Apr 2014 12:28:23 -0400 Subject: [PATCH] Removed another TODO about scrolling Areas on Windows (this time about aborting early if there seems to be no change). --- area_windows.go | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/area_windows.go b/area_windows.go index 151e7df..de429cc 100644 --- a/area_windows.go +++ b/area_windows.go @@ -342,10 +342,7 @@ func scrollArea(s *sysData, wparam _WPARAM, which uintptr) { newpos = maxsize - pagesize } - // TODO is this the right thing to do for SB_THUMBTRACK? or will it conflict? - if newpos == si.nPos { // no change; no scrolling - return - } + // this would be where we would put a check to not scroll if the scroll position changed, but see the note about SB_THUMBPOSITION above: Raymond Chen's code always does the scrolling anyway in this case delta := -(newpos - si.nPos) // negative because ScrollWindowEx() scrolls in the opposite direction dx := delta