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(textbox): skia textbox implementation #13901

Merged
merged 25 commits into from
Nov 8, 2023

Conversation

ramezgerges
Copy link
Contributor

@ramezgerges ramezgerges commented Oct 5, 2023

GitHub Issue (If applicable): #9417

PR Type

What kind of change does this PR introduce?

What is the current behavior?

What is the new behavior?

Copilot Summary

🤖 Generated by Copilot at acf7c2e

This pull request adds support for Unicode characters in key events on the Skia platform, by using platform-specific APIs to map keycodes to Unicode and adding a new UnicodeKey property to the KeyRoutedEventArgs and KeyEventArgs classes. It also adds a feature flag to allow switching between using a native overlay or a skia-based implementation for the TextBox control on the Skia platform, and fixes some layout, selection, and hyperlink issues for the TextBlock and TextBox controls.

PR Checklist

Please check if your PR fulfills the following requirements:

Other information

Internal Issue (If applicable):

@github-actions github-actions bot added the area/skia ✏️ Categorizes an issue or PR as relevant to Skia label Oct 5, 2023
@ramezgerges
Copy link
Contributor Author

Triple tapping is currently blocked by #13936

src/Uno.UI/UI/Xaml/Controls/TextBox/TextBox.cs Outdated Show resolved Hide resolved
src/Uno.UI/UI/Xaml/Controls/TextBox/TextBox.cs Outdated Show resolved Hide resolved
src/Uno.UI/UI/Xaml/Controls/TextBox/TextBox.cs Outdated Show resolved Hide resolved
src/Uno.UI/UI/Xaml/Controls/TextBox/TextBox.skia.cs Outdated Show resolved Hide resolved
src/Uno.UI/UI/Xaml/Controls/TextBox/TextBox.skia.cs Outdated Show resolved Hide resolved
src/Uno.UI/UI/Xaml/Documents/InlineCollection.skia.cs Outdated Show resolved Hide resolved
src/Uno.UI/UI/Xaml/Documents/InlineCollection.skia.cs Outdated Show resolved Hide resolved
@ramezgerges ramezgerges changed the title feat(textbox): single-line skia textbox implementation feat(textbox): skia textbox implementation Oct 20, 2023
@github-actions github-actions bot added the area/automation Categorizes an issue or PR as relevant to project automation label Oct 20, 2023
@ramezgerges ramezgerges force-pushed the skia_textbox_impl branch 2 times, most recently from 5b5104e to bbdd9be Compare October 26, 2023 09:02
Copy link
Member

@MartinZikmund MartinZikmund left a comment

Choose a reason for hiding this comment

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

Very nice!

@ramezgerges ramezgerges force-pushed the skia_textbox_impl branch 3 times, most recently from 4dd1ac5 to 5bcba64 Compare November 3, 2023 07:17
@ramezgerges
Copy link
Contributor Author

/azp run

Copy link

Azure Pipelines successfully started running 2 pipeline(s).

Copy link
Member

@jeromelaban jeromelaban left a comment

Choose a reason for hiding this comment

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

Let's move the resources to uno, even if temporarily, and we're good to go!

@github-actions github-actions bot added the platform/wasm 🌐 Categorizes an issue or PR as relevant to the WebAssembly platform label Nov 6, 2023
@ramezgerges ramezgerges force-pushed the skia_textbox_impl branch 2 times, most recently from 21869f4 to 0a929c1 Compare November 7, 2023 09:58
@jeromelaban jeromelaban merged commit e1a222f into unoplatform:master Nov 8, 2023
86 checks passed
@ramezgerges ramezgerges deleted the skia_textbox_impl branch December 6, 2023 17:58
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 area/skia ✏️ Categorizes an issue or PR as relevant to Skia 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.

4 participants