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
Export table scrolling methods #2317
Conversation
- ScrollTo, ScrollToEnd, ScrollToStart - addresses fyne-io#1892
Is End and Start best? It seems unlikely that bottom-right is more useful than bottom left? How about top and bottom like list with possible leading and trailing as well? |
So, what would leading and trailing do? scroll all the way left and all the way right, without changing the vertical offset? |
Yes I think so, if that makes sense? |
- these replace ScrollToStart() and ScrollToEnd()
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great addition, just a few comments on the exact functionality and comment/variable style
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good stuff, thanks again :)
Description:
This PR adds the following public methods to widget.Table:
Fixes #1892
Checklist:
I'm not certain about the public API style. The issue mentioned creating ScrollToTop() and ScrollToBottom() but a table is 2D, so I went with ScrollToStart() and ScrollToEnd(). Any thoughts on this?
I figured I'd get some feedback on this before working on scrolling for widget.Tree