Skip to content

Conversation

@PaulL1
Copy link
Contributor

@PaulL1 PaulL1 commented Sep 17, 2014

Including unit test extensions, and additions to tutorial 202.

Including unit test extensions, and additions to tutorial 202.
@swalters
Copy link
Contributor

Should this use renderContainer.visibleRowsCache and visibleColsCache instead of grid.Rows and grid.cols? If the data is filtered or rows are not visible, won't the calcs will be off?

@PaulL1
Copy link
Contributor Author

PaulL1 commented Sep 18, 2014

For columns, it's probably OK because I'm using drawnWidth. However, now you mention it I haven't dealt with RTL at all, nor pinned columns. I presume they would have some impact on the scrolling (definitely for RTL, maybe not materially for pinned containers).

For rows it will definitely impact.

In order to implement I need to understand something about render containers I think. How many render containers can there be, and how do I decide which render container to look in for the visible rows/cols cache to do the calculations? I'm assuming that pinned rows/cols would be in a separate render container?

@PaulL1
Copy link
Contributor Author

PaulL1 commented Sep 18, 2014

Superseded by #1546

@PaulL1 PaulL1 closed this Sep 18, 2014
@c0bra c0bra removed the in progress label Sep 18, 2014
@PaulL1 PaulL1 deleted the cellnav_scrollto branch September 26, 2014 08:32
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.

3 participants