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

OnIdiom doesn't respect Desktop setting on Windows #22093

Open
KeithBoynton opened this issue Apr 27, 2024 · 7 comments
Open

OnIdiom doesn't respect Desktop setting on Windows #22093

KeithBoynton opened this issue Apr 27, 2024 · 7 comments
Labels
platform/windows 🪟 s/triaged Issue has been reviewed t/bug Something isn't working
Milestone

Comments

@KeithBoynton
Copy link

Description

When using this for example:

TextColor="{OnIdiom Desktop='Red', Default='Black'}"

Running a solution on Windows the text is Black when I would expect it to be Red.

It works as expected when running on MacCatalyst, the text is Red.
It works as expected when running on iOS, the text is Black.
It works as expected when running on Android, the text is Black.
It does not work as expected when running on Windows, the text is Black.

Steps to Reproduce

1. Clone the attached project
2. Run the solution on MacCatalyst
3. Observe the text Red (as expected)
4. Run the solution on iOS
5. Observe the text Black (as expected)
6. Run the solution on Android
7. Observe the text Black (as expected)
8. Run the solution on Windows
9. Observe the text Black (NOT expected, Red was expected)

Link to public reproduction project repository

https://bitbucket.org/KeithBoynton/on-idiom-desktop-issue/src/master/

Version with bug

8.0.21 SR4.1

Is this a regression from previous behavior?

Not sure, did not test other versions

Last version that worked well

Unknown/Other

Affected platforms

Windows

Affected platform versions

No response

Did you find any workaround?

No

Relevant log output

No response

@KeithBoynton KeithBoynton added the t/bug Something isn't working label Apr 27, 2024
@ninachen03
Copy link
Collaborator

Verified this issue with Visual Studio 17.10.0 Preview 5 ( 8.0.21).I can not repro this issue.
image

@ninachen03 ninachen03 added s/triaged Issue has been reviewed s/try-latest-version Please try to reproduce the potential issue on the latest public version labels Apr 28, 2024
@KeithBoynton
Copy link
Author

Not working for me on Visual Studio 17.10.0 Preview 5 ( 8.0.21)

image

What am I missing?

@dotnet-policy-service dotnet-policy-service bot removed the s/try-latest-version Please try to reproduce the potential issue on the latest public version label Apr 28, 2024
@ninachen03
Copy link
Collaborator

ninachen03 commented Apr 29, 2024

We have verified that this problem, does not recur on 5 machines. We are not sure why it still reoccurs. Does it occur in a specific environment setting? In order for us to investigate this further, could you provide the Msbuild Log and binLog?

@jfversluis
Copy link
Member

Just tried this myself and same here, with your exact code, no changes, I see the text as red so I wonder what the difference is. Also tried a release build and same there.

Can you run dotnet --info and maybe add that info? Make sure you have installed the latest .NET 8 with the .NET MAUI workload?

@jfversluis jfversluis added the s/needs-info Issue needs more info from the author label May 1, 2024
@KeithBoynton
Copy link
Author

Please find attached build output and also the dotnet info. FYI I did a dotnet workload update but it's still the same.

dotnet-info.txt
OnIdiom-desktop-issue-build-output.txt

@dotnet-policy-service dotnet-policy-service bot added s/needs-attention Issue has more information and needs another look and removed s/needs-info Issue needs more info from the author labels May 3, 2024
@jfversluis jfversluis added this to the Backlog milestone May 6, 2024
@jfversluis jfversluis removed the s/needs-attention Issue has more information and needs another look label May 6, 2024
@jfversluis
Copy link
Member

Very strange and I don't have a good explanation for it for now unfortunately. This one seems to might be related? #21799

So maybe when that gets merged, that might fix this too

@KeithBoynton
Copy link
Author

Ok, I've subscribed to that issue.. however it's a draft and no activity on it for 3 weeks. What's blocking that being merged do we know?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
platform/windows 🪟 s/triaged Issue has been reviewed t/bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants