Carousel content overlaps bounds - second card only, iOS

[garnelsoftware]

Carousel content overlaps bounds - second card only, iOS

[AndreiMisiukevich]

Hi Tom, thanks for your warm words!

Can you try calling it carousel.ForceRedrawViews().
Also, can you build a small sample?

[garnelsoftware]

Hi Andrei, Great to hear from you, I shall try to get an example prepared. Many thanks, Tom

…

On Mon, 17 Jul 2023, 16:18 Andrei, ***@***.***> wrote: Hi Tom, thanks for your warm words! Can you try calling it carousel.ForceRedrawViews(). Also, can you build a small sample? — Reply to this email directly, view it on GitHub <#424 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AJCCEDBWJNI4MMPMSGEWJWLXQVJVRANCNFSM6AAAAAA2MVFCEY> . You are receiving this because you authored the thread.Message ID: ***@***.***>

[garnelsoftware]

Andrei, For a minute there I thought that had fixed it but I have at least narrowed it down to a more specific scenario. It's only happening when I use the "next" button which does only one thing: update the carousel selected index. If I swipe the control instead then the layout is correct! Sorry I've still not had time to prepare an example. Tom On Mon, 17 Jul 2023 at 16:31, Munchkin Studios < ***@***.***> wrote:

…

Hi Andrei, Great to hear from you, I shall try to get an example prepared. Many thanks, Tom On Mon, 17 Jul 2023, 16:18 Andrei, ***@***.***> wrote: > Hi Tom, thanks for your warm words! > > Can you try calling it carousel.ForceRedrawViews(). > Also, can you build a small sample? > > — > Reply to this email directly, view it on GitHub > <#424 (comment)>, > or unsubscribe > <https://github.com/notifications/unsubscribe-auth/AJCCEDBWJNI4MMPMSGEWJWLXQVJVRANCNFSM6AAAAAA2MVFCEY> > . > You are receiving this because you authored the thread.Message ID: > ***@***.***> >

[garnelsoftware]

I forgot to add that it's actually not restricted to landscape either. I hadn't realised that on a phone where I tended to use portrait I would swipe but on tablet, I'd use landscape and tended to use the buttons! On Tue, 18 Jul 2023, 09:18 Munchkin Studios, ***@***.***> wrote:

…

Andrei, For a minute there I thought that had fixed it but I have at least narrowed it down to a more specific scenario. It's only happening when I use the "next" button which does only one thing: update the carousel selected index. If I swipe the control instead then the layout is correct! Sorry I've still not had time to prepare an example. Tom On Mon, 17 Jul 2023 at 16:31, Munchkin Studios < ***@***.***> wrote: > Hi Andrei, > > Great to hear from you, I shall try to get an example prepared. > > Many thanks, > > Tom > > On Mon, 17 Jul 2023, 16:18 Andrei, ***@***.***> wrote: > >> Hi Tom, thanks for your warm words! >> >> Can you try calling it carousel.ForceRedrawViews(). >> Also, can you build a small sample? >> >> — >> Reply to this email directly, view it on GitHub >> <#424 (comment)>, >> or unsubscribe >> <https://github.com/notifications/unsubscribe-auth/AJCCEDBWJNI4MMPMSGEWJWLXQVJVRANCNFSM6AAAAAA2MVFCEY> >> . >> You are receiving this because you authored the thread.Message ID: >> ***@***.***> >> >

[AndreiMisiukevich]

@garnelsoftware hmm, interesting. You can try to implement your own arrows and use SelectedItem instead of SelectedIndex.

[garnelsoftware]

Thanks Andrei, After looking at the implementation of SelectedItem I feel that it will do the same as SelectedIndex. I have just tried calling carousel.OnSwiped(PanDirection) in place of the index modification and the same display glitch occurs... ForceRedrawViews caused the app to crash and so I abandoned that idea.

…

On Tue, 18 Jul 2023, 21:41 Andrei, ***@***.***> wrote: @garnelsoftware <https://github.com/garnelsoftware> hmm, interesting. You can try to implement your own arrows and use SelectedItem instead of SelectedIndex. — Reply to this email directly, view it on GitHub <#424 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AJCCEDHM72FZSKRPMQXVC7LXQ3YGRANCNFSM6AAAAAA2MVFCEY> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[AndreiMisiukevich]

@garnelsoftware try to call ForceRedrawViews in Main thread

[garnelsoftware]

I gave it a go but it made no difference. I best get that sample ready for you! Thanks for your help thus far!

…

On Wed, 19 Jul 2023, 15:59 Andrei, ***@***.***> wrote: @garnelsoftware <https://github.com/garnelsoftware> try to call ForceRedrawViews in Main thread — Reply to this email directly, view it on GitHub <#424 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AJCCEDAT552OZUQDF6RJP5LXQ7Y5NANCNFSM6AAAAAA2MVFCEY> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[garnelsoftware]

Andrei, I decided to take a step back and grab all of my view model bound height and widths in the constructor and store them in fields. Then when the layout method was invoked later in the constructor or upon rotate I could guarantee the only thing that would change would be the width/height of the page. This seems to have fixed my issue! I think what was happening is that the view model values were not always set when I expected them to and so sometimes the sizes would differ and throw the layout out of kilter. Sorry about all of that. Once again thank you for your excellent plugin AND thank you so much for providing me with support. Many thanks, Tom On Wed, 19 Jul 2023 at 21:22, Munchkin Studios < ***@***.***> wrote:

…

I gave it a go but it made no difference. I best get that sample ready for you! Thanks for your help thus far! On Wed, 19 Jul 2023, 15:59 Andrei, ***@***.***> wrote: > @garnelsoftware <https://github.com/garnelsoftware> try to call > ForceRedrawViews in Main thread > > — > Reply to this email directly, view it on GitHub > <#424 (comment)>, > or unsubscribe > <https://github.com/notifications/unsubscribe-auth/AJCCEDAT552OZUQDF6RJP5LXQ7Y5NANCNFSM6AAAAAA2MVFCEY> > . > You are receiving this because you were mentioned.Message ID: > ***@***.***> >

[AndreiMisiukevich]

Thanks Tom!