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.

Sign up for GitHub

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

Jump to bottom

[WASM][TextBox][PasswordBox] Control randomly not switching visual state when clicked #18040

Closed
eriklimakc opened this issue Aug 27, 2024 · 2 comments · Fixed by #18042
Closed

[WASM][TextBox][PasswordBox] Control randomly not switching visual state when clicked #18040

eriklimakc opened this issue Aug 27, 2024 · 2 comments · Fixed by #18042
Assignees
@Youssef1313
Labels
area/focus Categorizes an issue as relevant to focus management difficulty/tbd Categorizes an issue for which the difficulty level needs to be defined. kind/bug Something isn't working platform/wasm 🌐 Categorizes an issue or PR as relevant to the WebAssembly platform project/input ⌨️ Categorizes an issue or PR as relevant to input (Button, CheckBox, Toggle, Scroll, Map, Numeric,...) project/text 🔤 Categorizes an issue or PR as relevant to text (TextBox, PasswordBox, TextBlock, Fonts, …) triage/untriaged Indicates an issue requires triaging or verification

Comments

@eriklimakc
Copy link
Contributor

Current behavior

Sometimes when swiching the focus of TextBox / PasswordBox the control doesn't go to the right visual state.

repro.mp4

Expected behavior

No response

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

Repro: WasmVsBlank.zip

  1. Create a new uno app from the blank template
  2. Add TextBoxes / PasswordBoxes
  3. Play clicking them and observe

Workaround

No response

Works on UWP/WinUI

Yes

Environment

No response

NuGet package version(s)

No response

Affected platforms

No response

IDE

No response

IDE version

No response

Relevant plugins

No response

Anything else we need to know?

No response
@eriklimakc eriklimakc 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 Aug 27, 2024
@eriklimakc eriklimakc mentioned this issue Aug 27, 2024
Open
5 tasks
@Youssef1313 Youssef1313 self-assigned this Aug 27, 2024
@Youssef1313
Copy link
Member

When the issue manifests, it's a parent div that received the pointer event rather than the native input element. Issue is also still present in 5.4 devs where managed pointers implementation is used.

@Youssef1313 Youssef1313 added area/focus Categorizes an issue as relevant to focus management platform/wasm 🌐 Categorizes an issue or PR as relevant to the WebAssembly platform project/input ⌨️ Categorizes an issue or PR as relevant to input (Button, CheckBox, Toggle, Scroll, Map, Numeric,...) project/text 🔤 Categorizes an issue or PR as relevant to text (TextBox, PasswordBox, TextBlock, Fonts, …) labels Aug 27, 2024
@Youssef1313
Copy link
Member

Looks like a regression from #16982

@Youssef1313 Youssef1313 mentioned this issue Aug 27, 2024
Merged
6 tasks
Youssef1313 added a commit to Youssef1313/uno that referenced this issue Aug 28, 2024
@Youssef1313
test: Add test for unoplatform#18040
9eedbdb
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Youssef1313 Youssef1313

Labels
area/focus Categorizes an issue as relevant to focus management difficulty/tbd Categorizes an issue for which the difficulty level needs to be defined. kind/bug Something isn't working platform/wasm 🌐 Categorizes an issue or PR as relevant to the WebAssembly platform project/input ⌨️ Categorizes an issue or PR as relevant to input (Button, CheckBox, Toggle, Scroll, Map, Numeric,...) project/text 🔤 Categorizes an issue or PR as relevant to text (TextBox, PasswordBox, TextBlock, Fonts, …) triage/untriaged Indicates an issue requires triaging or verification
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(reg): Fix TextBox focus regression Youssef1313/uno
2 participants
@Youssef1313 @eriklimakc