Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[iOS] PipsPager breaks app layout #16715

Closed
2 of 16 tasks
mcocaroUX opened this issue May 7, 2024 · 7 comments
Closed
2 of 16 tasks

[iOS] PipsPager breaks app layout #16715

mcocaroUX opened this issue May 7, 2024 · 7 comments
Assignees
Labels
kind/bug Something isn't working

Comments

@mcocaroUX
Copy link

mcocaroUX commented May 7, 2024

Current behavior

Currently when having a PipsPager+FlipView linked by SelectorExtensions at a Page and its changes it selected value, safe area (bottom and top) turns black. I could reproduce this only on iOS
Before sliding:
Simulator_Screenshot_-iPhone_15-_2024-05-07_at_16 59 20
After sliding:
Simulator_Screenshot_-iPhone_15-_2024-05-07_at_16 59 39

Expected behavior

Safe area should not be affected

How to reproduce it (as minimally and precisely as possible)

FlipViewSafeArea.zip

Run sample
Flip to show next image
Notice how safe area turns black

Environment

Package Version(s):
Uno.Sdk: 5.2.108

<PackageVersion Include="Microsoft.Extensions.Logging.Console" Version="8.0.0" />
<PackageVersion Include="coverlet.collector" Version="6.0.0" />
<PackageVersion Include="FluentAssertions" Version="6.12.0" />
<PackageVersion Include="Microsoft.NET.Test.Sdk" Version="17.8.0" />
<PackageVersion Include="NUnit" Version="4.0.1" />
<PackageVersion Include="NUnit3TestAdapter" Version="4.5.0" />
<PackageVersion Include="Newtonsoft.Json" Version="13.0.3" />
<PackageVersion Include="Microsoft.Windows.SDK.BuildTools" Version="10.0.22621.2428" />
<PackageVersion Include="Microsoft.WindowsAppSDK" Version="1.4.231219000" />
<PackageVersion Include="Uno.Core.Extensions.Logging.Singleton" Version="4.0.1" />
<PackageVersion Include="Uno.Extensions.Hosting" Version="4.1.21" />
<PackageVersion Include="Uno.Extensions.Hosting.WinUI" Version="4.1.21" />
<PackageVersion Include="Uno.Extensions.Logging.OSLog" Version="1.7.0" />
<PackageVersion Include="Uno.Extensions.Logging.WinUI" Version="4.1.21" />
<PackageVersion Include="Uno.Extensions.Navigation" Version="4.1.21" />
<PackageVersion Include="Uno.Extensions.Navigation.WinUI" Version="4.1.21" />
<PackageVersion Include="Uno.Extensions.Navigation.Toolkit.WinUI" Version="4.1.21" />
<PackageVersion Include="Uno.Extensions.Reactive" Version="4.1.21" />
<PackageVersion Include="Uno.Extensions.Reactive.WinUI" Version="4.1.21" />
<PackageVersion Include="Uno.Material.WinUI" Version="5.0.13" />
<PackageVersion Include="Uno.Toolkit.WinUI" Version="6.0.18" />
<PackageVersion Include="Uno.Toolkit.WinUI.Material" Version="6.0.18" />
<PackageVersion Include="Uno.Resizetizer" Version="1.4.8" />
<PackageVersion Include="Uno.UI.Adapter.Microsoft.Extensions.Logging" Version="$(UnoVersion)" />
<PackageVersion Include="Uno.UniversalImageLoader" Version="1.9.36" />
<PackageVersion Include="Uno.WinUI" Version="$(UnoVersion)" />
<PackageVersion Include="Uno.WinUI.Lottie" Version="$(UnoVersion)" />
<PackageVersion Include="Uno.WinUI.DevServer" Version="$(UnoVersion)" />
<PackageVersion Include="Xamarin.Google.Android.Material" Version="1.10.0.2" />
<PackageVersion Include="Uno.UITest.Helpers" Version="1.1.0-dev.70" />
<PackageVersion Include="Xamarin.UITest" Version="4.3.3" />
  • iOS
  • macOS (AppKit)
  • Mac Catalyst
  • Android
  • WebAssembly
  • Windows
  • Skia (WPF)
  • Skia (GTK on Linux/macOS/Windows)
  • Skia (Linux Framebuffer)
  • Build tasks

Visual Studio:

  • 2019 (version: )
  • 2022 (version: )
  • Visual Studio Code (version: )
  • Rider Windows (version: )
  • Rider macOS (version: )

Relevant plugins:

  • Resharper (version: )

Anything else we need to know?

@mcocaroUX mcocaroUX added kind/bug Something isn't working triage/untriaged Indicates an issue requires triaging or verification labels May 7, 2024
@kazo0
Copy link
Contributor

kazo0 commented May 8, 2024

Looks to be the same/related to unoplatform/uno.toolkit.ui#1125

@kazo0
Copy link
Contributor

kazo0 commented May 14, 2024

This doesn't look to be related to SafeArea or SelectorExtensions. It seems to be an issue specific to PipsPager. If I create a simple all with just a PipsPager and attempt to change pips, I see the behavior:

MainPage.xaml

<Page x:Class="PipsApp.MainPage"
      xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
      xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
      xmlns:local="using:PipsApp"
      Background="{ThemeResource ApplicationPageBackgroundThemeBrush}">
      <Grid Background="Red">
            <PipsPager Background="Green"
                  HorizontalAlignment="Center"
                  Margin="0,50,0,0"
                  NumberOfPages="10"
                  x:Name="FlipViewPipsPager" />
      </Grid>
</Page>
Screen.Recording.2024-05-13.at.10.04.54.PM.mov

@kazo0 kazo0 transferred this issue from unoplatform/uno.toolkit.ui May 14, 2024
@kazo0 kazo0 changed the title [SelectorExtesions] [SafeArea] SafeArea turns black when using SelectorExtensions.PipsPager [iOS] PipsPager breaks app layout May 14, 2024
@kazo0
Copy link
Contributor

kazo0 commented May 14, 2024

PipsApp.zip

Simple repro app

@kazo0
Copy link
Contributor

kazo0 commented May 15, 2024

@mcocaroUX Would you be able to confirm that this is no longer occurring as of the Uno.Sdk version 5.3.0-dev.1256?

@mcocaroUX
Copy link
Author

@kazo0 Issue is fixed on 5.3.0-dev.1256. Thank you!

@agneszitte
Copy link
Contributor

@kazo0 do we need to backport something for 5.2 stable?

@jeromelaban
Copy link
Member

@agneszitte the changes implied cannot be backported, it's related to lifecycle changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants