dotnet / maui

.NET MAUI is the .NET Multi-platform App UI, a framework for building native device applications spanning mobile, tablet, and desktop.
https://dot.net/maui
MIT License
21.94k stars 1.7k forks source link

[regression/8.0.0-nightly] Nightly build 9725 broke boxviews do not get their color set anymore #19309

Closed Zack-G-I-T closed 8 months ago

Zack-G-I-T commented 8 months ago

Description

Using normal maui version, the boxviews that have a color attribute are shown correctly. However, when using maui nightly build 8.0.4-nightly.9717 - the boxviews do not have the correct color attributed to them.

I am debugging on a physical ios device using a windows PC.

I have attatched screenshots to show the boxviews. The first one is on the normal maui version - see how the big boxview is blue, but in the second one (using nightly build) it has no color.

image image image

Steps to Reproduce

  1. Download repo
  2. Run as normal - see the correct behaviour
  3. Change to use nightly build - see different behaviour

Link to public reproduction project repository

https://github.com/Zack-G-I-T/NightlyBuildIssue

Version with bug

Nightly / CI build (Please specify exact version)

Is this a regression from previous behavior?

Yes, this used to work in .NET MAUI

Last version that worked well

8.0.3

Affected platforms

iOS

Affected platform versions

No response

Did you find any workaround?

No response

Relevant log output

No response

gabsamples6 commented 8 months ago

we are also affected on this one and because we use Maui Community toolkit statecontainer to apply shimmering to our pages and the boxview is used all the pages where we apply the shimmering (loads of them) they look plain white .

It makes our app not suitable for production.

jsuarezruiz commented 8 months ago

@jfversluis Have done some tests on main branch and cannot reproduce the issue. Could you do some extra tests? image You can find the sample tests ready here https://github.com/dotnet/maui/tree/fix-19309

ghost commented 8 months ago

Hi @Zack-G-I-T. We have added the "s/needs-info" label to this issue, which indicates that we have an open question for you before we can take further action. 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.

jsuarezruiz commented 8 months ago

@jfversluis Have done some tests on main branch and cannot reproduce the issue. Could you do some extra tests? image You can find the sample tests ready here https://github.com/dotnet/maui/tree/fix-19309

Can reproduce it.

Redth commented 8 months ago

Looks like this regressed in: https://github.com/dotnet/maui/pull/18434

davidortinau commented 8 months ago

In case this matters, I'm on nightly 9742 and the first time I navigate to the page I see the box and then it disappears. I resize the window to no avail, but I can navigate away and back to see the BoxView once more. Mac Catalyst.

protected override void Build()
    {
        this.Title = "BoxViews";

        this.Content = new ScrollView{
            Content = new VerticalStackLayout{
                Spacing = 12,
                Children = {
                    new H1("100x100"),
                    new Separator(),
                    new BoxView{ Color = AppColors.Primary }.Size(100),       
                }
            }.DynamicResource(Label.StyleProperty, "MainContainer")
        };
    }
gabsamples6 commented 8 months ago

@PureWeen hi - glad to see this fixed as it affected us too, but could please advise which night build is this in? tried 9742 and we still have an issue

many thanks