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

Problem scrolling up with multiple items per row #93

Open
SaintFlipper opened this issue Jul 9, 2015 · 3 comments
Open

Problem scrolling up with multiple items per row #93

SaintFlipper opened this issue Jul 9, 2015 · 3 comments
Labels

Comments

@SaintFlipper
Copy link

Hi, this is the secondary issue raised as a side-effect in #89 ("No way to limit the data source top"). I've opened a new issue as it's really a different artifact and not related to negative indices.

If NGScroller is used to handling scrolling of multiple items per line its behaviour is confused when the user tries to scroll upwards:

  1. Trying to scroll upwards using the mouse wheel causes the scroll window to go down rather than up, ie. higher numbered items are displayed rather than lower items.
  2. Grabbing the scroll bar and dragging it to the top position usually results in out-of-order items and/or duplicate items.
    In this instance the scrollable items are DIVs with style display:block and float:left. There may be some small variation in behaviour between different browsers, but Chrome 43 on Windows seems fairly representative.

Standalone example at: http://jsfiddle.net/mn59dvjn/5/
Note: make the jsFiddle result pane wide enough to display at least 4 images per row to see the above effects.

@mfeingold
Copy link
Contributor

yea, looks like a bug. it similar to the problem with the inline elements. We have a problem with the setup when elements can jump rows as the width changes. I will mark it as a bug and we will try to fix it, in the meantime a possible solution would be to place everything going into a single row in a separate div.

On the side note we moved the code to angular-ui. All further development will be done in the angular-ui repo

@mfeingold mfeingold added the bug label Jul 17, 2015
@mfeingold mfeingold reopened this Jul 17, 2015
@dhilt
Copy link
Member

dhilt commented Jul 20, 2015

An issue was created in angular-ui repo: angular-ui/ui-scroll#8.

@mfeingold
Copy link
Contributor

@SaintFlipper I am not sure if this notification made it to you. I think we have a fix for the problem you reported here, but this fix is in the new repo. In general going forward all new development, including fixes will be in the new repo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants