Phoenix icon indicating copy to clipboard operation
Phoenix copied to clipboard

Apply padding in Parent Layout

Open desmond1121 opened this issue 10 years ago • 2 comments

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>

desmond1121 avatar Dec 15 '15 03:12 desmond1121

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 :(

shliama avatar Dec 15 '15 12:12 shliama

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

desmond1121 avatar Dec 15 '15 14:12 desmond1121