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

Update to .NET 8.0.60 broke custom (UraniumUI.Material) components rendering #23036

Closed
inimirpaz opened this issue Jun 13, 2024 · 13 comments
Closed
Assignees
Labels
area-layout StackLayout, GridLayout, ContentView, AbsoluteLayout, FlexLayout, ContentPresenter i/regression This issue described a confirmed regression on a currently supported version p/1 Work that is important, and has been scheduled for release in this or an upcoming sprint platform/android 🤖 platform/iOS 🍎 s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working
Milestone

Comments

@inimirpaz
Copy link

Description

Upgrading from 8.0.40 to 8.0.60 broke some components rendering (see repro and following screenshots)

.NET 8.0.40:
immagine

.NET 8.0.60:
immagine

Steps to Reproduce

No response

Link to public reproduction project repository

https://github.com/inimirpaz/maui_issues/tree/uraniumui_issue_net8_0_60

Version with bug

Unknown/Other

Is this a regression from previous behavior?

Yes, this used to work in .NET MAUI

Last version that worked well

8.0.40 SR5

Affected platforms

iOS, Android

Affected platform versions

No response

Did you find any workaround?

No response

Relevant log output

No response

@inimirpaz inimirpaz added the t/bug Something isn't working label Jun 13, 2024
Copy link
Contributor

Hi I'm an AI powered bot that finds similar issues based off the issue title.

Please view the issues below to see if they solve your problem, and if the issue describes your problem please consider closing this one and thumbs upping the other issue to help us prioritize it. Thank you!

Open similar issues:

Closed similar issues:

Note: You can give me feedback by thumbs upping or thumbs downing this comment.

@jsuarezruiz jsuarezruiz added potential-regression This issue described a possible regression on a currently supported version., verification pending area-layout StackLayout, GridLayout, ContentView, AbsoluteLayout, FlexLayout, ContentPresenter labels Jun 13, 2024
@PureWeen
Copy link
Member

@inimirpaz to clarify, I'm only seeing this not render on Android

image

@PureWeen PureWeen added i/regression This issue described a confirmed regression on a currently supported version and removed potential-regression This issue described a possible regression on a currently supported version., verification pending labels Jun 13, 2024
@PureWeen PureWeen self-assigned this Jun 13, 2024
@PureWeen PureWeen added this to the .NET 8 SR7 milestone Jun 13, 2024
@Zhanglirong-Winnie Zhanglirong-Winnie added s/verified Verified / Reproducible Issue ready for Engineering Triage s/triaged Issue has been reviewed labels Jun 14, 2024
@Zhanglirong-Winnie
Copy link
Collaborator

Verified this issue with Visual Studio 17.11.0 Preview 2.0 (8.0.60 & 8.0.40). Can repro on (8.0.60) Android and iOS platform with sample project. 8.0.40 works fins.

@inimirpaz
Copy link
Author

@inimirpaz to clarify, I'm only seeing this not render on Android
image

Hi, yes I am sorry, it is an Android-only issue indeed. I cannot edit the issue to remove the iOS tag it seems though

@phillippschmedt
Copy link

phillippschmedt commented Jun 14, 2024

We see the same issue in 8.0.60. It seems to be related to Frame. In my case I was able to replace Frame with Border to avoid this bug.

@PureWeen PureWeen modified the milestones: .NET 8 SR7, .NET 8 SR6 Jun 17, 2024
@PureWeen PureWeen added the p/1 Work that is important, and has been scheduled for release in this or an upcoming sprint label Jun 17, 2024
@PureWeen
Copy link
Member

@phillippschmedt do you have a repro you can attach by chance?

@dinisvieira
Copy link

dinisvieira commented Jun 18, 2024

@phillippschmedt do you have a repro you can attach by chance?

I can reproduce this just by having the MAUI Template App and adding the snippet below inside the StackLayout of the MainPage (non-shell) and using version 8.0.60 (version 8.0.40 works fine)

So, it seems the issue is having a Border inside a Frame on version 8.0.60 as far as I can tell.

<Frame>
	<StackLayout>
		<Border>
			<Label Text="Label Inside border and inside Frame"/>
		</Border>
		<Label Text="Label Outside border but inside Frame"/>
	</StackLayout>
</Frame>

@dinisvieira
Copy link

@PureWeen I created a repro in case it's useful.
https://github.com/dinisvieira/maui-frame-border-bug/tree/main

Not sure it's the exact issue that @phillippschmedt is having.

@phillippschmedt
Copy link

@PureWeen @dinisvieira Yes that is the same problem I experienced. Thank you.

@PureWeen
Copy link
Member

This fix should be available on the nightly feed as version 8.0.61-ci.net8.24319.3.

Can you test with the latest nightly build?
https://github.com/dotnet/maui/wiki/Nightly-Builds

@dinisvieira
Copy link

This fix should be available on the nightly feed as version 8.0.61-ci.net8.24319.3.

Can you test with the latest nightly build? https://github.com/dotnet/maui/wiki/Nightly-Builds

The Frame/Border issue seems to be fixed yes.

I did start getting the issue below on this nightly but it's likely something else.

**System.InvalidCastException:** 'Unable to convert instance of type 'android/graphics/drawable/BitmapDrawable' to type 'android.graphics.drawable.Animatable'.'

@PureWeen
Copy link
Member

PureWeen commented Jun 24, 2024

This fix should be available on the nightly feed as version 8.0.61-ci.net8.24319.3.
Can you test with the latest nightly build? https://github.com/dotnet/maui/wiki/Nightly-Builds

The Frame/Border issue seems to be fixed yes.

I did start getting the issue below on this nightly but it's likely something else.

**System.InvalidCastException:** 'Unable to convert instance of type 'android/graphics/drawable/BitmapDrawable' to type 'android.graphics.drawable.Animatable'.'

This should be fixed if you pull down 8.0.61 from nightly

@dinisvieira
Copy link

This fix should be available on the nightly feed as version 8.0.61-ci.net8.24319.3.
Can you test with the latest nightly build? https://github.com/dotnet/maui/wiki/Nightly-Builds

The Frame/Border issue seems to be fixed yes.
I did start getting the issue below on this nightly but it's likely something else.
**System.InvalidCastException:** 'Unable to convert instance of type 'android/graphics/drawable/BitmapDrawable' to type 'android.graphics.drawable.Animatable'.'

This should be fixed if you pull down 8.0.61 from nightly

I just tried it and both issues seem to be fixed.
Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-layout StackLayout, GridLayout, ContentView, AbsoluteLayout, FlexLayout, ContentPresenter i/regression This issue described a confirmed regression on a currently supported version p/1 Work that is important, and has been scheduled for release in this or an upcoming sprint platform/android 🤖 platform/iOS 🍎 s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working
Projects
Status: Done
Development

No branches or pull requests

7 participants