Flip is not show image #36

Closed
VAdaihiep opened this Issue Dec 27, 2012 · 4 comments

2 participants

@VAdaihiep

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
Owner

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

@openaphid openaphid was assigned Dec 29, 2012
@VAdaihiep

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
Owner

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

The behavior in previous version just happens to be correct.

@openaphid openaphid closed this Dec 31, 2012
@openaphid
Owner

#41 has been fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment