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

There is a rogue FluentUI.cs in Resources/Fonts #25928

Open
mattleibow opened this issue Nov 18, 2024 · 5 comments · May be fixed by #26036
Open

There is a rogue FluentUI.cs in Resources/Fonts #25928

mattleibow opened this issue Nov 18, 2024 · 5 comments · May be fixed by #26036
Labels
area-templates Project templates, Item Templates for Blazor and MAUI i/regression This issue described a confirmed regression on a currently supported version platform/android 🤖 platform/iOS 🍎 platform/macOS 🍏 macOS / Mac Catalyst platform/windows 🪟

Comments

@mattleibow
Copy link
Member

Description

I just did a dotnet new maui -f net9.0 and I see there is a file in the fonts folder: FluentUI.cs

Then I got this compiler error:

D:\GitHub\Testing\TestReadyToRunPublish\Resources\Fonts\FluentUI.cs(6,14): error CS0101: The namespace 'Fonts' already contains a definition for 'FluentUI'

when publishing Windows.

Steps to Reproduce

No response

Link to public reproduction project repository

No response

Version with bug

9.0.0 GA

Is this a regression from previous behavior?

Yes, this used to work in .NET MAUI

Last version that worked well

8.0.100 SR10

Affected platforms

Windows, macOS, Android, iOS, Other (Tizen, Linux, etc. not supported by Microsoft directly)

Affected platform versions

No response

Did you find any workaround?

No response

Relevant log output

@mattleibow mattleibow added the t/bug Something isn't working label Nov 18, 2024
@mattleibow mattleibow added this to the .NET 9 Servicing milestone Nov 18, 2024
@dotnet-policy-service dotnet-policy-service bot added the s/triaged Issue has been reviewed label Nov 18, 2024
@mattleibow mattleibow added i/regression This issue described a confirmed regression on a currently supported version area-templates Project templates, Item Templates for Blazor and MAUI and removed t/bug Something isn't working s/triaged Issue has been reviewed labels Nov 18, 2024
@jfversluis
Copy link
Member

Are you sure you're looking at the latest and greatest thing? Because dotnet new maui -f net9.0 will get you the template without sample data so that shouldn't have the FluentUI class at all, and according to my local testing it doesn't.

When I do dotnet new maui -f net9.0 --sample-content I do get the FluentUI class under the Resources/Fonts folder, but just doing a dotnet build does not give me any errors.

I guess you say when publishing to Windows. Through VS or through the command-line?

@jfversluis
Copy link
Member

In Windows, building and publishing seems to work fine for me

Image

@mattleibow
Copy link
Member Author

I'll check again, but I did it twice and I just got that file even without sample template.

@mattleibow
Copy link
Member Author

Image

mattleibow added a commit that referenced this issue Nov 21, 2024
This file was added for the fancy sample, but should be excluded as well if we are not fance

Fixes #25928
@mattleibow mattleibow linked a pull request Nov 21, 2024 that will close this issue
@mattleibow
Copy link
Member Author

I meant when not using the sample content that file still appears.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-templates Project templates, Item Templates for Blazor and MAUI i/regression This issue described a confirmed regression on a currently supported version platform/android 🤖 platform/iOS 🍎 platform/macOS 🍏 macOS / Mac Catalyst platform/windows 🪟
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants