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

Reset the Scrollbar position on the DataView Grid at the top when Filter is applied #8121

Merged

Conversation

N-thony
Copy link
Collaborator

@N-thony N-thony commented Feb 9, 2023

Fixes #8116
@rdstern @lloyddewit I made the change so that the scrollbar will always go at the top when the filter is applied. Have a look.

@N-thony N-thony changed the title Reset the Scrollbar on the DataView Grid position at the top when Filter is applied Reset the Scrollbar position on the DataView Grid at the top when Filter is applied Feb 9, 2023
@lloyddewit
Copy link
Contributor

@berylwaswa if you can test, then we can merge, thanks

@rdstern
Copy link
Collaborator

rdstern commented Feb 10, 2023

@N-thony while you are here could you check the same thing "sideways". I had 34 columns and only saw 30. When I was at Variable 30 I went to the second page it was blank. I think it is ok if I am at variable 1.

I have just confirmed with a blank data frame of 35 variables (if the limit is 30). If I am on Variable x1 and the move to the next page, all is fine.

But if I am on x30, then the next page appears blank. It is ok once I press in the blank data frame. But again it would be good to set back to the first variable visible, before changing the page.

Copy link
Collaborator

@rdstern rdstern left a comment

Choose a reason for hiding this comment

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

@N-thony I confirm that the scrolling vertically seems fixed. I then looked at scrolling horizontally with a new data frame of width 54 (my number of variables is set to 50) and hope the same can be done for this?
If you prefer it to be another pull request, then I'll approve this one.

@N-thony
Copy link
Collaborator Author

N-thony commented Feb 13, 2023

The solution here fixes the issue but it doesn't solve the problem in the Grid in general because the same problem found with the columns is also with the rows. I will try to fix it more general with the Grid.

Copy link
Collaborator

@rdstern rdstern left a comment

Choose a reason for hiding this comment

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

I am approving, because the sideways problem can be in a separate pull request. And it would be good to have this feature fixed in time for the March workshops

@lloyddewit lloyddewit added the bug label Feb 24, 2023
@lloyddewit lloyddewit merged commit f75d685 into IDEMSInternational:master Feb 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Scrolling problem when data filtered
3 participants