Skip to content

fix: Horizontal Scroll Causes Complete Overflow#54

Merged
kubeden merged 2 commits intointerpretive-systems:mainfrom
Terminal239:bug/horizontal-scroll-fix
Oct 3, 2025
Merged

fix: Horizontal Scroll Causes Complete Overflow#54
kubeden merged 2 commits intointerpretive-systems:mainfrom
Terminal239:bug/horizontal-scroll-fix

Conversation

@Terminal239
Copy link
Copy Markdown
Contributor

@Terminal239 Terminal239 commented Oct 2, 2025

Summary

Made some changes in the rendering functions to fix the scroll overflow issue.

  • Misplaced variable in the sliceANSI function causing complete removal of content string.
  • renderSideCell cleaned to only have the slice to only have the slicing be done once directly using the content.
  • rightBodyLinesAll cleaned to have the padding applied regardless of rightXOffset.

Previously:

bdd7-3286-4441-af3f-f22ec74c1863.mp4

Fix:

5909-b9ae-4f9a-91f5-6eeea4dc3bd6.mp4

Linked Issues

Fixes: #8

Checklist

  • I am assigned to the linked issue(s)
  • I wrote tests or updated existing tests
  • I ran go build and go test ./...
  • I updated docs/README if needed

@kubeden kubeden self-requested a review October 3, 2025 13:07
@kubeden kubeden merged commit 90acf2f into interpretive-systems:main Oct 3, 2025
3 of 15 checks passed
@kubeden
Copy link
Copy Markdown
Contributor

kubeden commented Oct 3, 2025

Good job :) merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[bug] long lines overflow & truncate = not visible

2 participants