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

feat: EnterKeyHint #13336

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

feat: EnterKeyHint #13336

wants to merge 5 commits into from

Conversation

MartinZikmund
Copy link
Member

@MartinZikmund MartinZikmund commented Aug 21, 2023

GitHub Issue (If applicable): closes #13347

PR Type

What kind of change does this PR introduce?

  • Feature

What is the current behavior?

Missing support

What is the new behavior?

Support for EnterKeyHint in WebAssembly.

Copilot Summary

🤖 Generated by Copilot at d25b09d

This pull request adds support for the EnterKeyHint property to the TextBox control on WebAssembly, which allows specifying the text and behavior of the enter key on the virtual keyboard. It also adds a new sample page to the SamplesApp project and a new section to the documentation article to demonstrate the usage of the property. Additionally, it refactors some existing code to use the nameof operator for dependency property names.

PR Checklist

Please check if your PR fulfills the following requirements:

Other information

Internal Issue (If applicable):

@MartinZikmund MartinZikmund self-assigned this Aug 21, 2023
@github-actions github-actions bot added platform/wasm 🌐 Categorizes an issue or PR as relevant to the WebAssembly platform platform/android 🤖 Categorizes an issue or PR as relevant to the Android platform platform/ios 🍎 Categorizes an issue or PR as relevant to the iOS platform area/automation Categorizes an issue or PR as relevant to project automation kind/documentation labels Aug 21, 2023
#elif __ANDROID__
MyTextBox.ImeOptions = ImeAction.Search;
#elif __WASM__
MyTextBox.EnterKeyHint = EnterKeyHint.Search;
Copy link
Member

@jeromelaban jeromelaban Aug 22, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be be creating an attached property that includes this behavior instead? After all, this is Search everywhere and we could hide platform specifics from the user.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like Jerome's suggestion. It will highly simplify the usage.

Copy link

This PR is stale because it has been open 60 days with no activity. Remove stale label or comment or it will be closed in 10 days.

@github-actions github-actions bot added the stale This item has been marked as stale and will be closed if there is no activity. label Jan 30, 2025
@MartinZikmund MartinZikmund removed the stale This item has been marked as stale and will be closed if there is no activity. label Jan 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/automation Categorizes an issue or PR as relevant to project automation kind/documentation platform/android 🤖 Categorizes an issue or PR as relevant to the Android platform platform/ios 🍎 Categorizes an issue or PR as relevant to the iOS platform platform/wasm 🌐 Categorizes an issue or PR as relevant to the WebAssembly platform
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for EnterKeyHint on WebAssembly
4 participants