From 38b7126449ab046a4a56633d859a89e1cc9faee7 Mon Sep 17 00:00:00 2001
From: Pietro Gagliardi <pietro10@mac.com>
Date: Thu, 11 Dec 2014 20:35:46 -0500
Subject: [PATCH] Fixed up the whole horizontal scroll glitching mess. Now to
 tackle the twitchy text...

---
 wintable/new/header.h | 15 +++++++++++++++
 wintable/new/main.c   |  1 -
 2 files changed, 15 insertions(+), 1 deletion(-)

diff --git a/wintable/new/header.h b/wintable/new/header.h
index f23c812..d0fe457 100644
--- a/wintable/new/header.h
+++ b/wintable/new/header.h
@@ -56,10 +56,15 @@ static void headerAddColumn(struct table *t, WCHAR *name)
 		panic("error adding column to Table header");
 }
 
+// TODO make a better name for this?
+// TODO move to hscroll.h?
+// TODO organize this in general...
+// TODO because of this function's new extended functionality only hscrollto() is allowed to call repositionHeader()
 static void updateTableWidth(struct table *t)
 {
 	HDITEMW item;
 	intptr_t i;
+	RECT client;
 
 	t->width = 0;
 	// TODO count dividers?
@@ -70,6 +75,16 @@ static void updateTableWidth(struct table *t)
 			panic("error getting Table column width for updateTableWidth()");
 		t->width += item.cxy;
 	}
+
+	if (GetClientRect(t->hwnd, &client) == 0)
+		panic("error getting Table client rect in updateTableWidth()");
+	t->hpagesize = client.right - client.left;
+
+	// this part is critical: if we resize the columns to less than the client area width, then the following hscrollby() will make t->hscrollpos negative, which does very bad things
+	// note to self: do this regardless of whether the table width or the client area width was changed
+	if (t->hpagesize > t->width)
+		t->hpagesize = t->width;
+
 	// do a dummy scroll to update the horizontal scrollbar to use the new width
 	hscrollby(t, 0);
 }
diff --git a/wintable/new/main.c b/wintable/new/main.c
index 7d3c87e..e2f85d9 100644
--- a/wintable/new/main.c
+++ b/wintable/new/main.c
@@ -26,7 +26,6 @@
 // TODO
 // - should tablePanic be CALLBACK or some other equivalent macro? and definitely export initTable somehow, but which alias macro to use?
 // - make panic messages grammatically correct ("Table error: adding...")
-// - figure out how to handle the situation where we resize the table width to less than the width of the client rect...
 
 #define tableWindowClass L"gouitable"