search

etsy / AndroidStaggeredGrid

An Android staggered grid view which supports multiple columns with rows of varying sizes.
https://github.com/etsy/AndroidStaggeredGrid
4.76k stars 1.13k forks source link

Fix the adapter index to getItemId() to include the mFirstPosition offse... #86

Closed davejohncole closed 10 years ago

davejohncole commented 10 years ago

I noticed that the row id being passed to onItemClick() was not being offset from the top visible item in the grid. I tracked the problem down to the PerformClick class where I saw that the offset being applied to the position was not being applied to the index that is used to retrieve the row id.

denizmveli commented 10 years ago

@davejohncole thx for the fix!