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

[WASM] BreadcrumbBar isn't responsive #12483

Open
morning4coffe-dev opened this issue May 31, 2023 · 9 comments
Open

[WASM] BreadcrumbBar isn't responsive #12483

morning4coffe-dev opened this issue May 31, 2023 · 9 comments
Assignees
Labels
area/items-repeater Categorizes an issue or PR as relevant the ItemsRepeater control difficulty/challenging 🤯 Categorizes an issue for which the difficulty level is reachable with internals understanding kind/bug Something isn't working project/layout 🧱 Categorizes an issue or PR as relevant to layouting and containers (Measure/Arrange, Collections,..)

Comments

@morning4coffe-dev
Copy link
Member

morning4coffe-dev commented May 31, 2023

Current behavior

Recording.2023-06-01.004318.mp4

Expected behavior

No response

How to reproduce it (as minimally and precisely as possible)

http://playground.platform.uno/#aa5e4903

image

  1. Resize the window

or

  1. Go to Gallery
  2. Look for BreadcrumbBar
  3. Change the window size

Workaround

No response

Works on UWP/WinUI

Yes

Environment

Uno.UI / Uno.UI.WebAssembly / Uno.UI.Skia

Uno.WinUI / Uno.WinUI.WebAssembly / Uno.WinUI.Skia

NuGet package version(s)

No response

Affected platforms

WebAssembly

IDE

No response

IDE version

No response

Relevant plugins

No response

Anything else we need to know?

No response

@morning4coffe-dev morning4coffe-dev added kind/bug Something isn't working triage/untriaged Indicates an issue requires triaging or verification difficulty/tbd Categorizes an issue for which the difficulty level needs to be defined. labels May 31, 2023
@jeromelaban jeromelaban added project/layout 🧱 Categorizes an issue or PR as relevant to layouting and containers (Measure/Arrange, Collections,..) and removed triage/untriaged Indicates an issue requires triaging or verification labels Jun 1, 2023
@jeromelaban
Copy link
Member

If this does not happen with WinUI, we may not be having the latest sources?

@morning4coffe-dev
Copy link
Member Author

@jeromelaban i have tested it on WinUI project also and I am getting the same issue. I have updated this with playground link.

@jeromelaban
Copy link
Member

@morning4coffe-dev when you're saying you tested on WinUI, you mean a WinAppSDK project?

@morning4coffe-dev
Copy link
Member Author

@jeromelaban yes 👍

@jeromelaban
Copy link
Member

I see, so the section in the original issue saying works on UWP: Yes was incorrect. This would mean that it's not an issue, but rather a behavior from the official implementation of the control. /cc @MartinZikmund

@MartinZikmund
Copy link
Member

@jeromelaban that was a misunderstanding, it does work correctly on WinUI, it does not on Uno, so the issue is valid

@Arieldelossantos Arieldelossantos self-assigned this Aug 18, 2023
@Arieldelossantos
Copy link

Arieldelossantos commented Aug 24, 2023

hey @MartinZikmund / @kazo0 i've been working with this issue and i noticed that the issue is reproducible on the Wasm Sample app on Uno.UI when screen is resized to smaller width than the breadCrumBar content... so the method that is hit is MeasureOverride...

image

image

image

the issue is that the first node (Root >) keeps visible

the weird part is that if we add another node, the first node hides and the behavior works as espected

image

any idea on which other method i should be looking on when resizing screen???

@MartinZikmund
Copy link
Member

It should definitely happen somewhere withing the Measure/Arrange phase, but it is quite possible that the root cause will be hidden somewhere within the ItemsRepeater code, which is quite complex...

@Arieldelossantos
Copy link

as you mentioned @MartinZikmund the issue apparently is coming from the itemRepeater and the Arrange method from the UIElement class :/

@MartinZikmund MartinZikmund added area/items-repeater Categorizes an issue or PR as relevant the ItemsRepeater control difficulty/challenging 🤯 Categorizes an issue for which the difficulty level is reachable with internals understanding and removed difficulty/tbd Categorizes an issue for which the difficulty level needs to be defined. labels Sep 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/items-repeater Categorizes an issue or PR as relevant the ItemsRepeater control difficulty/challenging 🤯 Categorizes an issue for which the difficulty level is reachable with internals understanding kind/bug Something isn't working project/layout 🧱 Categorizes an issue or PR as relevant to layouting and containers (Measure/Arrange, Collections,..)
Projects
None yet
Development

No branches or pull requests

4 participants