search

microsoft / fluentui-blazor

Microsoft Fluent UI Blazor components library. For use with ASP.NET Core Blazor applications
https://www.fluentui-blazor.net
MIT License
3.3k stars 305 forks source link

FluentComboBox and FluentSelect don't work with mouse selection in Microsoft.Maui.Controls v8.0.14 #1741

Open euantorano opened 2 months ago

euantorano commented 2 months ago

🐛 Bug Report

When upgrading Microsoft.Maui.Controls and/or Microsoft.Maui.Controls.Compatibility, the <FluentComboBox> and <FluentSelect> stop accepting mouse inputs to select items. Keyboard actions to select items still work.

Microsoft.Maui.Controls > 8.0.7 causes this bug. Microsoft.Maui.Controls.Compatibility > 8.0.7 causes this bug.

Using older versions of these libraries (e.g. using 8.0.7 for both) resolves the problem.

Note that other components may also be broken, but I have observed both FLuentCombobox and FluentSelect myself.

💻 Repro or Code Sample

See repository: https://github.com/euantorano/fluentui-combo-and-select

Branch main does not work, but the branch working-version does work.

🤔 Expected Behavior

Selecting items with the mouse should work for both FluentComboBox and FluentSelect.

😯 Current Behavior

Selecting items with the mouse does not work.

7tvOT60LIM

💁 Possible Solution

N/A - apart from downgrading dependencies

🔦 Context

N/A

🌍 Your Environment

jamesmontemagno commented 2 months ago

@eilon did we change something in the blazor hybrid web view that folks need to specify the render type? do folks need interactivity enabled somehow?

subratb commented 2 months ago

Seems like this issue has resurfaced again. When we had the issue earlier, we solved it by adding a package reference to a particular version of Microsoft.WindowsAppSDK(1.4.231115000). But, now we have the same issue again.

mohit-x0 commented 2 months ago

I am also facing the same issue, selecting items <FluentSelect> with the mouse does not work on Windows, but it works on Android.

vnbaaij commented 2 months ago

It seems to be a regression on the MAUI side. We are blocked till it is fixed there.

jem-experience commented 1 month ago

Oh thank you, -1 in my backlog, i had the same issue for weeks after a big UI redesign from pure html css to fluentui components, i was thinking at a css thing so i push myself really hard to delay the investigation, i have others issues, with no css involved, to manage! I solved it by downgrading the Microsoft.Maui.Essentials from 8.0.21 to version="8.0.6" and the 2 others previously mentionned and everything is fine now!

euantorano commented 3 weeks ago

Looks like this is still broken in v8.0.21. Is there any timeline for a fix on the MAUI side or a link to an issue on that side of things?

vnbaaij commented 3 weeks ago

You should really ask that in the MAUI repo. We are not on that team and do not have any insights into their planning/roadmap

euantorano commented 3 weeks ago

OK. When you came to the conclusion in https://github.com/microsoft/fluentui-blazor/issues/1741#issuecomment-2032702895, what made you realise it was a regression on their side? When I open an issue over there I want to provide as much detail as I can.

vnbaaij commented 3 weeks ago

Based on the fact that it worked fine with version 8.0.6 or 7 of the Microsoft.Maui.Componets library.

euantorano commented 3 weeks ago

OK, thanks. Issue opened with the MAUI team: https://github.com/dotnet/maui/issues/22297

Eilon commented 3 weeks ago

This appears to be a bug in WinUI WebView2 specifically. I opened a MAUI-less repro bug here: https://github.com/MicrosoftEdge/WebView2Feedback/issues/4545