From bd45fd655c1314660349e009e761d1b0abb68fa6 Mon Sep 17 00:00:00 2001
From: jpbruyere <jp.bruyere@hotmail.com>
Date: Wed, 27 Jan 2016 14:01:18 +0100
Subject: [PATCH] test refresh of scrollbar in listbox template, add
 RegisterForGraphicUpdate in scroller props

---
 Tests/GOLIBTest_Listbox.cs         | 13 ++++++--
 Tests/Interfaces/test_Listbox.goml | 48 ++++++++++++++++--------------
 src/GraphicObjects/Scrollbar.cs    |  2 ++
 3 files changed, 38 insertions(+), 25 deletions(-)

diff --git a/Tests/GOLIBTest_Listbox.cs b/Tests/GOLIBTest_Listbox.cs
index 1d6b2f38..cedf695a 100644
--- a/Tests/GOLIBTest_Listbox.cs
+++ b/Tests/GOLIBTest_Listbox.cs
@@ -82,13 +82,21 @@ namespace test
 		{
 			base.OnLoad (e);
 
-			TestList = Directory.GetFileSystemEntries("/home/jp/tmp/mtgdata/a", "*.txt",SearchOption.AllDirectories).ToList();
+			//TestList = Directory.GetFileSystemEntries("/home/jp/tmp/mtgdata/a", "*.txt",SearchOption.AllDirectories).ToList();
 			GraphicObject tlb = LoadInterface("Interfaces/test_Listbox.goml");
 			tlb.DataSource = this;
 
 //			TestList [1].Field = "test string";
 //			ValueChanged.Raise(this, new ValueChangeEventArgs ("TestList", TestList));
 		}
+		void OnClear (object sender, MouseButtonEventArgs e){
+			TestList = null;
+			ValueChanged.Raise(this, new ValueChangeEventArgs ("TestList", TestList));
+		}
+		void OnLoadList (object sender, MouseButtonEventArgs e){
+			TestList = Directory.GetFileSystemEntries("/home/jp/tmp/mtgdata/a", "*.txt",SearchOption.AllDirectories).ToList();
+			ValueChanged.Raise(this, new ValueChangeEventArgs ("TestList", TestList));
+		}
 		protected override void OnMouseMove (MouseMoveEventArgs e)
 		{			
 			base.OnMouseMove (e);
@@ -102,7 +110,8 @@ namespace test
 		}
 		protected override void OnKeyDown (KeyboardKeyEventArgs e)
 		{
-			TestList.Add ("newly added list item");
+			TestList = Directory.GetFileSystemEntries("/home/jp/tmp/mtgdata/a", "*.txt",SearchOption.AllDirectories).ToList();
+			//TestList.Add ("newly added list item");
 			ValueChanged.Raise(this, new ValueChangeEventArgs ("TestList", TestList));
 		}
 
diff --git a/Tests/Interfaces/test_Listbox.goml b/Tests/Interfaces/test_Listbox.goml
index 342f2909..fa4b730c 100755
--- a/Tests/Interfaces/test_Listbox.goml
+++ b/Tests/Interfaces/test_Listbox.goml
@@ -1,25 +1,27 @@
 <?xml version="1.0"?>
 <!--<ListBox Data="{TestList}" Focusable="true"/>-->
-<VerticalStack Fit="true">
-	<Label Text="{MousePos}" Width="-1" Background="DarkGreen"/>
-	<Label Text="{Hover}" Width="-1" Background="DarkGreen"/>
-	<Button Text="button 1" Width="150"/>
-	<ListBox Data="{TestList}" Background="0,5;0,5;0,5;0,5"
-				 HorizontalAlignment="Center" Width="-1" Height="150" Margin="5">
-		<Template>
-			<Border BorderWidth="1" MinimumSize="20;20" Height="{../TemplatedHeight}" Width="{../TemplatedWidth}">
-				<HorizontalStack Margin="1" Height="{../../TemplatedHeight}" Width="{../../TemplatedWidth}">
-					<Scroller Name="scroller1" Height="{../../../TemplatedHeight}" Width="{../../../TemplatedWidth}" 
-							Margin="2" VerticalScrolling="true" ScrollY="{../scrollbar1.Scroll}">
-						<VerticalStack Width="{../../../../TemplatedWidth}" Height="-1" 
-							Name="List" Margin="0" VerticalAlignment="Top" />
-					</Scroller>
-					<Scrollbar Name="scrollbar1" Scroll="{../scroller1.ScrollY}" 
-						MaximumScroll="{../scroller1.MaximumScroll}" Orientation="Vertical" 
-						Width="10" Height="{../../../TemplatedHeight}" />
-				</HorizontalStack>
-			</Border>
-		</Template>
-	</ListBox>
-	<Button Text="button 1" Width="0"/>
-</VerticalStack>
+<Border BorderWidth="2" Fit="true">
+	<VerticalStack Fit="true" Margin="50">
+		<Label Text="{MousePos}" Width="-1" Background="DarkGreen"/>
+		<Label Text="{Hover}" Width="-1" Background="DarkGreen"/>
+		<Button Text="Clear" Width="150" MouseClick="OnClear"/>
+		<ListBox Data="{TestList}" Background="0,5;0,5;0,5;0,5"
+					 HorizontalAlignment="Center" Width="-1" Height="150" Margin="5">
+			<Template>
+				<Border BorderWidth="1" Height="{../TemplatedHeight}" Width="{../TemplatedWidth}">
+					<HorizontalStack Margin="1" Height="{../../TemplatedHeight}" Width="{../../TemplatedWidth}" >
+						<Scroller Name="scroller1" Height="{../../../TemplatedHeight}" Width="{../../../TemplatedWidth}" 
+								Margin="2" VerticalScrolling="true" ScrollY="{../scrollbar1.Scroll}">
+							<VerticalStack Width="{../../../../TemplatedWidth}" Height="-1" MinimumSize="100;100"
+								Name="List" Margin="0" VerticalAlignment="Top" />
+						</Scroller>
+						<Scrollbar Name="scrollbar1" Scroll="{../scroller1.ScrollY}" 
+							MaximumScroll="{../scroller1.MaximumScroll}" Orientation="Vertical" 
+							Width="10" Height="{../../../TemplatedHeight}" />
+					</HorizontalStack>
+				</Border>
+			</Template>
+		</ListBox>
+		<Button Text="Load list" Width="0" MouseClick="OnLoadList"/>
+	</VerticalStack>
+</Border>
\ No newline at end of file
diff --git a/src/GraphicObjects/Scrollbar.cs b/src/GraphicObjects/Scrollbar.cs
index 8e3b2db5..956d96ab 100644
--- a/src/GraphicObjects/Scrollbar.cs
+++ b/src/GraphicObjects/Scrollbar.cs
@@ -46,6 +46,7 @@ namespace go
 				if (_maximumScroll == value)
 					return;
 				_maximumScroll = value;
+				registerForGraphicUpdate ();
 				ValueChanged.Raise(this, new ValueChangeEventArgs ("MaximumScroll", _maximumScroll));
 			}
 		}
@@ -74,6 +75,7 @@ namespace go
 					return;
 				_orientation = value;
 				ValueChanged.Raise(this, new ValueChangeEventArgs ("Orientation", _orientation));
+				registerForGraphicUpdate ();
 			}
 		}
 		public void onScrollBack (object sender, MouseButtonEventArgs e)
-- 
2.47.3