Merge pull request #2310 from AnkushJadhav/fix/2266

Fix: hidden table seperators being rendered incorrectly
fyne-io · Jul 7, 2021 · 0431028 · 0431028
2 parents 8f342e0 + 7f59b82
commit 0431028
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 16 deletions.
diff --git a/widget/table.go b/widget/table.go
@@ -373,7 +373,7 @@ func (t *tableRenderer) moveIndicators() {
 	}
 
 	for i := divs; i < len(t.dividers); i++ {
-		t.dividers[divs].Hide()
+		t.dividers[i].Hide()
 	}
 	canvas.Refresh(t.t)
 }

diff --git a/widget/testdata/table/desktop/hovered.xml b/widget/testdata/table/desktop/hovered.xml
@@ -42,12 +42,6 @@
 			<widget pos="4,79" size="168x1" type="*widget.Separator">
 				<rectangle fillColor="disabled" size="168x1"/>
 			</widget>
-			<widget size="0x0" type="*widget.Separator">
-				<rectangle fillColor="disabled" size="0x0"/>
-			</widget>
-			<widget size="0x0" type="*widget.Separator">
-				<rectangle fillColor="disabled" size="0x0"/>
-			</widget>
 		</widget>
 	</content>
 </canvas>
diff --git a/widget/testdata/table/desktop/hovered_out.xml b/widget/testdata/table/desktop/hovered_out.xml
@@ -31,15 +31,6 @@
 			<widget pos="4,41" size="168x1" type="*widget.Separator">
 				<rectangle fillColor="disabled" size="168x1"/>
 			</widget>
-			<widget size="0x0" type="*widget.Separator">
-				<rectangle fillColor="disabled" size="0x0"/>
-			</widget>
-			<widget size="0x0" type="*widget.Separator">
-				<rectangle fillColor="disabled" size="0x0"/>
-			</widget>
-			<widget size="0x0" type="*widget.Separator">
-				<rectangle fillColor="disabled" size="0x0"/>
-			</widget>
 		</widget>
 	</content>
 </canvas>