openaphid / android-flip

A component for flip animation on Android, which is similar to the effect in Flipboard iPhone/Android
http://openaphid.github.com/
2.84k stars 1.25k forks source link

Flip is not show image #36

Closed VAdaihiep closed 11 years ago

VAdaihiep commented 11 years ago

Hi openaphid and everyone, I'm loading some image from internet using a lib https://github.com/novoda/ImageLoader like your Demo: Flip Async Content. The issue is when flip, the image is not show. I guess the cause is flipView take screenshot before the image load complete. But I dont know how to fix this. So please fix this or suggest me something to do this. I already try refreshPage() but it doesn't effect. 2012-12-27_17-34-27 Load complete image

2012-12-27_17-32-08 But when flip: image is gone.

openaphid commented 11 years ago

Please check out the last update in develop branch. It's much better for pages with async content

VAdaihiep commented 11 years ago

Thank you, your update works very well. But I have a new small issue with new FlipViewController: In your demo FlipButtonActivity, I modified to add more button when flip complete at position near Adapter.getCount()

flipView.setOnViewFlipListener(new ViewFlipListener() {
            @Override
            public void onViewFlipped(View view, int position) {
                if(position > mAdapter.getCount() - 3)
                    mAdapter.addMoreButton();
            }
});

in class Adapter:

    int size = 5;

    public void addMoreButton(){
        size += 5;
    }

    @Override
    public int getCount() {
        return size;
    }

The issue is: When flip to page 4, I can't flip to page 5. But I think mAdapter already added page 5 because when set MAX_TIP_ANGLE = 90; I see number 5. (hope you understand, my english is not good, I can post a picture if you want)

Note that older versions of FlipViewController isn't have that isssue. So please help me to fix this issue to add dynamic page into Adapter. Thank you and happy new year!

openaphid commented 11 years ago

I created a new issue (#41) to address the problem.

The behavior in previous version just happens to be correct.

openaphid commented 11 years ago

41 has been fixed