-
Notifications
You must be signed in to change notification settings - Fork 447
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
Horizontal scrolling no longer works as expected #51
Comments
I just upgraded to the latest version and ran into this issue as well. I'm currently using the grid as a single horizontal row of cells. I would prefer this as an option, rather than always forcing the grid to scroll in both directions. |
I saw that, too. I removed the addition of the pixel and made a comment in my copy of the code. |
+1 |
Just ran into this issue as well and the behavior runs directly counter to what one would expect when using the grid view in a horizontal only setup. +1 |
+1 do not work |
It used to be that the horizontal scrolling wouldn't allow vertical scrolling. In many cases, this is desirable behavior. For example, if you have a grid that only scrolls horizontally in one orientation but allows vertical and horizontal scrolling when you move to another orientation.
Since the commit listed below, though, the content size of the grid is always forced to be one more pixel high than the grid size, meaning the grid will always scroll vertically, even if you don't want it to. I'm not sure why this commit was made or what problem it was solving, but it seems like an unnecessary thing to force. If anything, it should be available as a configurable option. Though I think it should be dropped altogether:
1e4fe3d#Classes/AQGridView.m
The text was updated successfully, but these errors were encountered: