[GH-ISSUE #1084] table.SetWrapSelection: may not wrap backwards in some cases #788

New issue

Closed

opened 2026-03-04 01:07:45 +03:00 by kerem · 1 comment

kerem commented

2026-03-04 01:07:45 +03:00

Owner

Originally created by @rfwatson on GitHub (Apr 10, 2025).
Original GitHub issue: https://github.com/rivo/tview/issues/1084

Hello,
In #842 it was reported that table selection wrapping was broken when the table contained non-selectable rows or columns e.g. a header.
This was fixed in 6785f5f but it seems that the behaviour is still partially present.

In a vertical wraparound where row 0 is non-selectable:

when pressing down on the last item of the table, the selection correctly wraps to row 1
when pressing up on row 1, the table view tries to select row 0 and does not wrap around, this seems to be a bug

This is recreatable in the original example provided in #842.

Originally created by @rfwatson on GitHub (Apr 10, 2025). Original GitHub issue: https://github.com/rivo/tview/issues/1084 Hello, In #842 it was reported that table selection wrapping was broken when the table contained non-selectable rows or columns e.g. a header. This was fixed in 6785f5f but it seems that the behaviour is still partially present. In a vertical wraparound where row 0 is non-selectable: * when pressing down on the last item of the table, the selection correctly wraps to row 1 * when pressing up on row 1, the table view tries to select row 0 and does not wrap around, this seems to be a bug This is recreatable in the original example provided in #842.

kerem closed this issue

2026-03-04 01:07:45 +03:00

kerem commented

2026-03-04 01:07:46 +03:00

Author

Owner

@rivo commented on GitHub (Aug 27, 2025):

I'm not sure why I thought this was resolved. Anyway, I tested this example after the linked commit and it worked.

@rivo commented on GitHub (Aug 27, 2025): I'm not sure why I thought this was resolved. Anyway, I tested this example after the linked commit and it worked.