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

.NET 8 #1367

Merged
merged 29 commits into from
Nov 4, 2023
Merged

.NET 8 #1367

merged 29 commits into from
Nov 4, 2023

Conversation

VladislavAntonyuk
Copy link
Collaborator

@VladislavAntonyuk VladislavAntonyuk commented Aug 24, 2023

Description of Change

Linked Issues

PR Checklist

Additional information

@brminnick
Copy link
Collaborator

@VladislavAntonyuk FYI - the PR we merged today introduced merge conflicts in the csproj files on this PR

@brminnick
Copy link
Collaborator

I wonder if we can also remove our reference to the DotNet.ReproducibleBuilds NuGet package now that SourceLink is included in .NET 8: https://learn.microsoft.com/en-us/dotnet/core/whats-new/dotnet-8#source-link?

I haven't tried it yet with the toolkit, but it's certainly something we can look into!

@VladislavAntonyuk
Copy link
Collaborator Author

I wonder if we can also remove our reference to the DotNet.ReproducibleBuilds NuGet package now that SourceLink is included in .NET 8: learn.microsoft.com/en-us/dotnet/core/whats-new/dotnet-8#source-link?

I haven't tried it yet with the toolkit, but it's certainly something we can look into!

removed, let's see how it works. we can always add it if necessary

@brminnick
Copy link
Collaborator

Update: @CliffAgius is going to update our documentation on how to consume our preview packages so that community members can use this .NET 8 pre-release of CommunityToolkit.Maui:

  1. Update the Wiki
  2. Add documentation to the Official Docs: https://learn.microsoft.com/en-us/dotnet/communitytoolkit/maui/

@inforithmics
Copy link
Contributor

inforithmics commented Sep 8, 2023

Any plans to include this

#1342 (comment)

.Net Maui has now the possibility to AddLogicalChildren instead of only Setting the Parent.

@brminnick brminnick added breaking change This label is used for PRs that include a breaking change and removed needs discussion Discuss it on the next Monthly standup labels Oct 3, 2023
@ne0rrmatrix
Copy link
Contributor

ne0rrmatrix commented Oct 28, 2023

Media Element issue's and dotnet 8

On IOS the player position is not being reported. Works fine on Windows. Tested on RC2.

Probably the numeric validation bug is the issue here too. I have also noticed some odd behavior with media element with dotnet 8.

It works fine if you use built in controls and do nothing else. Trying to get position in code behind and events are not working properly. They do not fire on Android or IOS. Everything works great on windows though.

edit: This build does work fine in toolkit sample build and appears only to be an issue for my use case. I am having media element events not firing and I am unable to track position of media playback when using custom controls. I will have to investigate further. But media playback with built in controls appears to track playback position just fine. Events for media element are untested except for not explicitly working for me. This is most likely an implementation issue on my end though. It works fine on 7.0x but again I want to reiterate it builds and works fine as part of toolkit.

@ne0rrmatrix
Copy link
Contributor

What is the best practices for reporting issues with this PR? I am having issues with differences in behavior between devices? Should I even mention it? The official samples work as expected when I test the feature I am having issues with. Should I even be commenting with bugs? Or should I wait till its released?

@brminnick brminnick enabled auto-merge (squash) November 4, 2023 23:22
Copy link
Collaborator

@brminnick brminnick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Vlad!!

azure-pipelines.yml Outdated Show resolved Hide resolved
azure-pipelines.yml Outdated Show resolved Hide resolved
azure-pipelines.yml Show resolved Hide resolved
azure-pipelines.yml Show resolved Hide resolved
global.json Outdated Show resolved Hide resolved
@brminnick
Copy link
Collaborator

Thanks @ne0rrmatrix!

Now that we've merged our PR supporting .NET 8, could you test MediaElement again on the main branch and then open an Issue if you confirm the bug still exists?

@brminnick brminnick merged commit d92d3c6 into main Nov 4, 2023
7 checks passed
@brminnick brminnick deleted the net8 branch November 4, 2023 23:31
@ne0rrmatrix
Copy link
Contributor

I tested my use case against main. I am having issues on Android and IOS. Untested on Mac atm. I don't believe the issue is caused by media element itself though. The sample builds fine and works as expected.

I am having issues with being unable to get the position of playback while playing any media. I can get the duration just fine. I am using a content view and have created custom controls. The content view is unable to get the position. It works fine in dotnet 7.x app but as soon as I switch to 8.x it stops working. This does not appear to be an issue for windows at all. It is just mobile devices like android or IOS. Have not tested on mac. It only affects me when using a content view. Directly using media element everything works as expected.

The issue is most likely not directly related to the toolkit. It appears to be an issue with either my code implementation of binding or a bug in Maui itself. I will have to do further testing to figure out where it is the problem originates.

@jkotas jkotas mentioned this pull request Apr 23, 2024
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking change This label is used for PRs that include a breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] Snackbar InvalidOperationException when using dotnet trimmer [Proposal] .Net 8 Preview (Beta) Nuget
4 participants