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

Project using MultiPage migrated from Xamarin faults on MAUI - Resubmission #6024

Closed
david-maw opened this issue Apr 12, 2022 · 7 comments
Closed
Labels
legacy-area-controls Label, Button, CheckBox, Slider, Stepper, Switch, Picker, Entry, Editor migration-compatibility Xamarin.Forms to .NET MAUI Migration, Upgrade Assistant, Try-Convert p/2 Work that is important, but is currently not scheduled for release platform/windows 🪟 s/triaged Issue has been reviewed s/try-latest-version Please try to reproduce the potential issue on the latest public version t/bug Something isn't working
Milestone

Comments

@david-maw
Copy link

Description

This is a resubmission of issue #5622 as requested by @Redth on that issue which he'd theorized in the comments on that issue would be fixed by some CarouselView fixes.

Note, this IMPLEMENTS a quick and dirty CarouselPage replacement, it does not use a CarouselView.

Steps to Reproduce

See #5622

Version with bug

Release Candidate 1

Last version that worked well

Unknown/Other

Affected platforms

Windows, I was not able test on other platforms

Affected platform versions

Windows 10

Did you find any workaround?

No

Relevant log output

No response

@david-maw david-maw added s/needs-verification Indicates that this issue needs initial verification before further triage will happen t/bug Something isn't working labels Apr 12, 2022
@Eilon Eilon added the legacy-area-controls Label, Button, CheckBox, Slider, Stepper, Switch, Picker, Entry, Editor label Apr 13, 2022
@jfversluis jfversluis added area/migration 🚚 and removed s/needs-verification Indicates that this issue needs initial verification before further triage will happen labels Apr 14, 2022
@jsuarezruiz jsuarezruiz added this to the 6.0.300-rc.3 milestone Apr 18, 2022
@Redth Redth modified the milestones: 6.0.300-rc.3, 6.0.300 Apr 27, 2022
@davidortinau davidortinau added the p/1 Work that is important, and has been scheduled for release in this or an upcoming sprint label Apr 30, 2022
@Redth Redth added p/2 Work that is important, but is currently not scheduled for release and removed p/1 Work that is important, and has been scheduled for release in this or an upcoming sprint labels May 3, 2022
@VincentBu
Copy link

not repro with vs main build(32622.314)
@david-maw is this the expected result?
image

@VincentBu VincentBu added the s/triaged Issue has been reviewed label Jun 23, 2022
@david-maw
Copy link
Author

@VincentBu

The original issue #5622 shows how it is meant to look and has an essentially identical Xamarin project for illustration. That said, the page heading you showed looks ok except for the rather weird oversize "-" between the "+" and "X". The Xamarin version does not show the page title ("Page 1" in your example) but that smells like a Xamarin bug. You can't see the page title in the Screen snapshot I included with #5622 because the frame rate counter covers that area- just run the Xamarin app without a debugger attached and the frame rate counter will not be enabled, revealing that area of the page.

@alexanderSP11
Copy link

Should this work on mobile? It started on windows, but there is a crash on android or ios.

@david-maw
Copy link
Author

Yes, it was intended to work on Mobile too (the Xamarin example does, on Android at least).

@mattleibow mattleibow modified the milestones: 6.0.300, 6.0-servicing Aug 29, 2022
@Redth Redth modified the milestones: 6.0-servicing, Backlog Aug 30, 2022
@ghost
Copy link

ghost commented Aug 30, 2022

We've moved this issue to the Backlog milestone. This means that it is not going to be worked on for the coming release. We will reassess the backlog following the current release and consider this item at that time. To learn more about our issue management process and to have better expectation regarding different types of issues you can read our Triage Process.

@samhouts samhouts removed the s/triaged Issue has been reviewed label Apr 5, 2023
@samhouts samhouts added migration-compatibility Xamarin.Forms to .NET MAUI Migration, Upgrade Assistant, Try-Convert and removed area/migration 🚚 labels Aug 28, 2023
@Zhanglirong-Winnie Zhanglirong-Winnie added s/triaged Issue has been reviewed s/try-latest-version Please try to reproduce the potential issue on the latest public version labels Nov 20, 2023
@ghost
Copy link

ghost commented Nov 20, 2023

Hi @david-maw. We have added the "s/try-latest-version" label to this issue, which indicates that we'd like you to try and reproduce this issue on the latest available public version. This can happen because we think that this issue was fixed in a version that has just been released, or the information provided by you indicates that you might be working with an older version.

You can install the latest version by installing the latest Visual Studio (Preview) with the .NET MAUI workload installed. If the issue still persists, please let us know with any additional details and ideally a reproduction project provided through a GitHub repository.

This issue will be closed automatically in 7 days if we do not hear back from you by then - please feel free to re-open it if you come back to this issue after that time.

@Zhanglirong-Winnie
Copy link

Verified this issue with Visual Studio Enterprise 17.9.0 Preview 1.0. Not repro on android platform with sample project.
Carousel-CheapCarouselPage-Xamarin.zip
Screenshot 2023-11-20 150153

@ghost ghost closed this as completed Nov 27, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Dec 28, 2023
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
legacy-area-controls Label, Button, CheckBox, Slider, Stepper, Switch, Picker, Entry, Editor migration-compatibility Xamarin.Forms to .NET MAUI Migration, Upgrade Assistant, Try-Convert p/2 Work that is important, but is currently not scheduled for release platform/windows 🪟 s/triaged Issue has been reviewed s/try-latest-version Please try to reproduce the potential issue on the latest public version t/bug Something isn't working
Projects
None yet
Development

No branches or pull requests