CommunityToolkit / Windows

Collection of controls for WinUI 2, WinUI 3, and Uno Platform developers. Simplifies and demonstrates common developer tasks building experiences for Windows with .NET.
https://aka.ms/windowstoolkitdocs
Other
553 stars 71 forks source link

Graphical glitches and poor performance on ColorPicker under WebAssembly #385

Open Arlodotexe opened 6 months ago

Arlodotexe commented 6 months ago

Describe the bug

The ColorPicker component was added in https://github.com/CommunityToolkit/Windows/pull/241 and recently released in 8.1-rc.

We had run into some minor issues ColorPicker on wasm before, but this was related to the Animations component (see here).

ColorPicker appears to work fine on Uwp and WinAppSDK, but has these graphical glitches on Wasm.

Steps to reproduce

1. Open toolkitlabs.dev or build and deploy locally
2. Open the colorpicker component in the browser
3. Try to use it. Observe the visual bugs.

Expected behavior

No visual bugs or choppy performance.

Screenshots

https://github.com/CommunityToolkit/Windows/assets/9384894/0c1b1a7d-71cb-4047-a0f7-e2ba51675386

Code Platform

Windows Build Number

Other Windows Build number

No response

App minimum and target SDK version

Other SDK version

No response

Visual Studio Version

No response

Visual Studio Build Number

No response

Device form factor

No response

Additional context

No response

Help us help you

Yes, but only if others can assist.

michael-hawker commented 6 months ago

Thanks for testing this Arlo, do we think it's related to the number of calls the component makes as the user drags their mouse or something some potential bottleneck in Uno/WASM?

michael-hawker commented 6 months ago

On my devbox I'm not noticing any perf issues (though it's new), but I did see the flickering.

I also see the grid missing always: image

And the bars that display here didn't appear the first time I opened that segment: image

Arlodotexe commented 2 months ago

Linking https://github.com/unoplatform/uno/issues/5992

michael-hawker commented 2 months ago

We should update to the latest Uno version for the next release now and see what gaps still exist.