Skip to content

[footer] column aggregation doesn't work when the column is not in the visible part (horizontal) grid's scroll #3456

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

Open
prdn opened this issue May 8, 2015 · 5 comments

Comments

@prdn
Copy link

prdn commented May 8, 2015

How to reproduce the issue:

  1. go here http://ui-grid.info/docs/#/tutorial/401_AllFeatures
  2. before clicking refreshing data
  3. scroll to the horizontal right end (so that the avg column aggregation is not visible)
  4. click on refresh button and wait until complete
  5. move to the horizontal left end to display the avg column and it is empty

Here is a screenshot of the result
uigrid

@PaulL1
Copy link
Contributor

PaulL1 commented May 16, 2015

I wonder if this is related to #3442, and again there's an instance where the column is initially not visible, so the aggregation doesn't get called from ui-grid-column. Then when we scroll so that it is visible it doesn't run the aggregation.

This one I'd fix by always doing an initial updateAggregation when the column is rendered.

@YonatanKra
Copy link
Contributor

I can't reproduce this case. It works fine with current unstable version (in chrome) as far as I can see. Do you still experience this? In what browser?

@PaulL1
Copy link
Contributor

PaulL1 commented Jun 2, 2015

I may have fixed this as part of fixing another related issue with aggregations. 5265115

@YonatanKra
Copy link
Contributor

So is this one closed?

@mweltin
Copy link

mweltin commented Nov 6, 2017

This still appears to be an issue even in v4.0.8

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

No branches or pull requests

5 participants