setPageMargin() does not work

[Mariovc]

The margins set on setPageMargin() method are ignored and overrided by MultiViewPager

[pflammertsma]

Correct; MultiViewPager adjusts the margin itself for the mechanism through which it centers pages. I would suggest specifying the margin through the page's layout padding instead.

…

On Thu, Aug 10, 2017, 18:28 Mario Velasco Casquero ***@***.***> wrote: The margins set on setPageMargin() method are ignored and overrided by MultiViewPager — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#22>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AARdY4Bv2yhzVTAX1ko7ci5Bowbg3OLvks5sWy-UgaJpZM4OzrEY> .

[Mariovc]

Putting margin to the view is a trick to "solve" it, but that is no the correct way for two reasons:

1. The method setPageMargin() doesn't work
2. The margins will be animated together with the image. For example scaling the page x2 will create a double space between elements.

[pflammertsma]

Those are good points. However, I don't plan on updating this library as I instead recommend moving forward with LinearSnapHelper in conjunction with RecyclerViews.

[Mariovc]

Thanks for the point out. Could I have animations with it?

[Mariovc]

In the Readme there is a receference to LinearSnapHelper but I think it is not a good recplacement if you need any of the following requisites:

• Animation when turning pages.
• Bullet points. An indicator to show on which page you are.
• Pages are turned one by one without acceleration.
• Auto-scroll, pages are turned automatically.

Maybe it not a good idea to point out to LinearSnapHelper from the Readme file without taking this into account