Yalantis / Phoenix

Phoenix Pull-to-Refresh
https://yalantis.com/
Apache License 2.0
4.02k stars 920 forks source link

Apply padding in Parent Layout #24

Open desmond1121 opened 8 years ago

desmond1121 commented 8 years ago

I've noticed that you handle bottom padding in scroll back. That would cause much time in "measure". I catched a systrace of your "scroll back":

trace

I understand why you need this logic, View.offsetTopAndBottom would destroy the bottom padding. But this not handled in scroll down yet. Actually, you can notify user to set padding in parent layout such as following code, and handle padding logic in onLayout() of PullToRefreshView.

<com.yalantis.phoenix.PullToRefreshView
        android:id="@+id/pull_to_refresh"
        android:padding="10dp"
        android:layout_width="match_parent"
        android:layout_height="match_parent">

    <ListView
        android:id="@+id/yalantis_list_view"
        android:divider="@null"
        android:dividerHeight="0dp"
        android:layout_width="match_parent"
        android:layout_height="match_parent" />
</com.yalantis.phoenix.PullToRefreshView>
shliama commented 8 years ago

Thanks for reporting the issue. I'll look into it this week and try to find a workaround because it would be hard to ask every library user to update their layouts :(

desmond1121 commented 8 years ago

Yes, it is. You can just take padding property value by content's attributes, and apply it in your it in your onLayout() :)