Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

[Core] Shapes not rendering changing Shell tabs #11927

Closed
wants to merge 9 commits into from
Closed

Conversation

jsuarezruiz
Copy link
Contributor

@jsuarezruiz jsuarezruiz commented Aug 26, 2020

Description of Change

Shapes not rendering changing Shell tabs.

Issues Resolved

API Changes

None

Platforms Affected

  • Core/XAML (all platforms)

Behavioral/Visual Changes

None

Before/After Screenshots

fix11924-ios
fix11924-droid

Testing Procedure

Launch Core Gallery and navigate to the issue 11924. Navigate to the second Tab and then navigate back to the first tab, and verify if the Ellipse is rendering.

PR Checklist

  • Targets the correct branch
  • Tests are passing (or failures are unrelated)

@martinkabraham
Copy link

@jsuarezruiz .Tried with Nuget 4.8.0.1269 pre-release in Xamarin Shell. Not Working while Tab is selected from second time. First time it is working.

Below shape I placed in second tab ShapePage when I navigate to AboutPage and again back to ShapePage ellipse not rendering.

<FlyoutItem FlyoutDisplayOptions="AsMultipleItems"> <ShellContent Title="About" Icon="tab_about.png" Route="AboutPage" ContentTemplate="{DataTemplate local:AboutPage}" /> <ShellContent Title="Browse" Icon="tab_feed.png" ContentTemplate="{DataTemplate local:ShapePage}" /> </FlyoutItem>

<Ellipse Fill="DarkBlue" Stroke="Red" StrokeThickness="4" WidthRequest="150" HeightRequest="50" HorizontalOptions="Start" />

@martinkabraham
Copy link

xamarin

@samhouts samhouts added the retarget-branch-required PR or associated issues target a milestone. Please target this PR to the matching branch. label Aug 26, 2020
@samhouts samhouts changed the base branch from 4.8.0 to 5.0.0 August 27, 2020 14:37
@samhouts samhouts added retarget-branch-required PR or associated issues target a milestone. Please target this PR to the matching branch. and removed retarget-branch-required PR or associated issues target a milestone. Please target this PR to the matching branch. labels Aug 27, 2020
@samhouts samhouts removed this from In Review in vCurrent (4.8.0) Aug 27, 2020
@samhouts samhouts added this to In Review in vNext+1 (5.0.0) Aug 27, 2020
@jsuarezruiz
Copy link
Contributor Author

Thanks for the feedback @martinkabraham. Will do more tests. Testing with https://github.com/xamarin/Xamarin.Forms/blob/34bbf5d6fbf920a16717686766f220e13348cf0b/Xamarin.Forms.Controls.Issues/Xamarin.Forms.Controls.Issues.Shared/Issue11924.cs the Ellipse is always rendering (I did the change of tabs more than 10 times).

@samhouts samhouts removed the retarget-branch-required PR or associated issues target a milestone. Please target this PR to the matching branch. label Aug 31, 2020
@samhouts samhouts added retarget-branch-required PR or associated issues target a milestone. Please target this PR to the matching branch. and removed retarget-branch-required PR or associated issues target a milestone. Please target this PR to the matching branch. labels Nov 2, 2020
@PureWeen PureWeen added this to In progress in v5.0.1 via automation Nov 5, 2020
@PureWeen PureWeen removed this from In Review in vNext+1 (5.0.0) Nov 5, 2020
@PureWeen PureWeen added this to the 5.0.1 milestone Nov 5, 2020
@samhouts samhouts added this to In Review in vNext+1 (5.0.0) Nov 5, 2020
@PureWeen PureWeen removed this from In progress in v5.0.1 Nov 5, 2020
@PureWeen PureWeen added this to In progress in v5.0.1 via automation Nov 5, 2020
@PureWeen PureWeen removed this from In Review in vNext+1 (5.0.0) Nov 5, 2020
@Cfun1
Copy link

Cfun1 commented Nov 24, 2020

There is a case where it is still happening, the case where you have 3 pages inside a Tab with condition that the shapes are contained in Page3:

Steps to reproduce:

1- Define the below Shell pages structure.
2- Navigate from Page1 to Page3.
3- Navigate to Page1 then come back to Page3

Version with issue: 5.0.0-pre4

  <FlyoutItem FlyoutDisplayOptions="AsMultipleItems">
        <Tab Title="Tab1">
            <ShellContent Title="Page1"
                          ContentTemplate="{DataTemplate local:Parent}"/>

            <ShellContent Title="Page2"
                          ContentTemplate="{DataTemplate local:Page2}"/>

            <ShellContent Title="Page3"
                          ContentTemplate="{DataTemplate local:Page3}"/> <!-- Contains shapes -->
        </Tab>
    </FlyoutItem>

@jsuarezruiz
Copy link
Contributor Author

I'm going to close this PR because this fix is now in #13284

v5.0.1 automation moved this from In progress to Done Jan 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.