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

Horizontal scrolling no longer works as expected #51

Open
drewmca opened this issue Mar 10, 2011 · 5 comments
Open

Horizontal scrolling no longer works as expected #51

drewmca opened this issue Mar 10, 2011 · 5 comments

Comments

@drewmca
Copy link

drewmca commented Mar 10, 2011

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

@lightandshadow68
Copy link

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.

@marksuman
Copy link

I saw that, too. I removed the addition of the pixel and made a comment in my copy of the code.

@myell0w
Copy link
Contributor

myell0w commented Jul 13, 2011

+1

@postmechanical
Copy link

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

@scinfu
Copy link

scinfu commented Dec 27, 2011

+1 do not work

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

No branches or pull requests

6 participants