Android-ObservableScrollView icon indicating copy to clipboard operation
Android-ObservableScrollView copied to clipboard

fix scrollY when the GridView has vertical spacing

Open jaredrummler opened this issue 8 years ago • 1 comments

The scrollY is off when items have vertical spacing. This fixes the scrollY jumping to incorrect values while scrolling.

This will still be an issue on pre-JB. If you did want to fix this problem for API 11-16, you could use reflection to get the vertical spacing. Example:


private Integer mVerticalSpacing;

@Override public int getVerticalSpacing() {
  if (VERSION.SDK_INT >= VERSION_CODES.JELLY_BEAN) {
    return super.getVerticalSpacing();
  }
  if (mVerticalSpacing == null) {
    try {
      Field field = GridView.class.getDeclaredField("mVerticalSpacing");
      if (!field.isAccessible()) {
        field.setAccessible(true);
      }
      mVerticalSpacing = field.getInt(this);
    } catch (Exception e) {
      mVerticalSpacing = 0;
    }
  }
  return mVerticalSpacing;
}

jaredrummler avatar Jun 14 '16 12:06 jaredrummler

Coverage Status

Coverage increased (+0.02%) to 93.384% when pulling 3abb0504676e9837f278312ede516e9fcaf992b8 on jaredrummler:master into 47a5fb2db5e93d923a8c6772cde48bbb7d932345 on ksoichiro:master.

coveralls avatar Jun 14 '16 13:06 coveralls