Yalantis / Euclid

User Profile Interface Animation
https://yalantis.com
2.23k stars 553 forks source link

java.lang.IllegalArgumentException: View must be inside RevealFrameLayout or RevealLinearLayout. #1

Closed xiaomeixw closed 9 years ago

xiaomeixw commented 9 years ago

java.lang.IllegalArgumentException: View must be inside RevealFrameLayout or RevealLinearLayout.

xiaomeixw commented 9 years ago

I know the way to solve the bug : overlay_list_item.xml need change like this:

<?xml version="1.0" encoding="utf-8"?>
<io.codetail.widget.RevealFrameLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:tools="http://schemas.android.com/tools"
    android:orientation="vertical"
    android:layout_width="match_parent"
    android:layout_height="wrap_content"
    android:background="@color/gray">

    <include
        layout="@layout/list_item"
        android:id="@+id/wrapper_list_item" />

    <LinearLayout
        android:id="@+id/wrapper_list_item_reveal"
        android:layout_width="match_parent"
        android:layout_height="match_parent"
        android:layout_alignTop="@+id/wrapper_list_item"
        android:layout_alignBottom="@+id/wrapper_list_item"
        android:orientation="vertical"
        android:visibility="invisible"
        android:background="@color/gray">

        <ImageView
            android:id="@+id/image_view_reveal_avatar"
            android:layout_width="match_parent"
            android:layout_height="@dimen/height_profile_image"
            android:clickable="true"
            tools:src="@drawable/nicolas_cage" />

    </LinearLayout>

</io.codetail.widget.RevealFrameLayout>
xiaomeixw commented 9 years ago

Thanks for your project,but i think you can make it more Beautiful than the version1.0; I think should to improve the Code reuse too; it‘s good project,thanks for your Sharism;

shliama commented 9 years ago

Thanks for your contribution :+1: My fault, forgot to merge changes for pre-L devices...

Couple fixes, including yours were added into master branch (ノ◕ヮ◕)ノ*:・゚✧

P.S. Yep, it's just a pilot version of the project, we published it on github because why not?)) I will continue updating it in future.