Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(scroll): scroll pane by actual amount of rows #1025

Merged
merged 1 commit into from
Jan 26, 2022

Conversation

tlinford
Copy link
Contributor

@tlinford tlinford commented Jan 26, 2022

use pane content_rows to take account of frame enabled/disabled

fix #1002

use pane content_rows to take account of frame enabled/disabled
Copy link
Member

@jaeheonji jaeheonji left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've seen issues related to this before. So I tested it through several programs (bat-cat, btop). The result is perfect 😄

I think this can solve this issue as well (#952)

@tlinford
Copy link
Contributor Author

tlinford commented Jan 26, 2022

I think this can solve this issue as well (#952)

Unfortunately that's another problem, but I'm making progress :)

@jaeheonji
Copy link
Member

jaeheonji commented Jan 26, 2022

Unfortunately that's another problem

Oh.. your right, it's my mistake. It looked similar, but it must be a different issue. After running some tests again, I found that it was definitely different. thank you 😅

@tlinford tlinford merged commit a576b31 into main Jan 26, 2022
@tlinford tlinford deleted the fix/scroll-page-off-by-one branch January 26, 2022 17:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Zellij skips one extra line with Page-{Up,Down} when frames are enabled
2 participants