Skip to content

More Text Input Support (#8672) Recommit #8740

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

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

Cn-mjt44
Copy link

Fix Event handling.

Fixes #

The text input now trigger by WM_CHAR and WM_UNICHAR

Description of Change

  • Add IsInputingText, IMEPosition and ClipboardText to abstract class GameWindow
  • Add Text Editing event and Text Input event.
    Event \ Platform WinodwsDX DesktopGL / VK
    Text Input Invoked by WM_CHAR or WM_UNICHAR Invoked by SDL_TEXTINPUT
    Text Editing Invoked by WM_IME_COMPOSITION or WM_IME_ENDCOMPOSITION Invoked by SDL_TEXTEDITING

@Cn-mjt44
Copy link
Author

@SimonDarksideJ This is recommit PR for More Text Input Support (#8672)

@SimonDarksideJ
Copy link
Contributor

SimonDarksideJ commented Apr 29, 2025

Can you please check your commits @Cn-mjt44 , we do not accept PR's with Formatting changes unless the file in question is actually being modified. For instance, BoundingBox has no actual changes other than formatting.

Please only commit files that have actually been altered.

Additionally, this fails the "Console Check" which validates AoT compatibility required for console support and the Native pipeline, features such as reflection are not permitted.

@Cn-mjt44
Copy link
Author

@SimonDarksideJ I have push this in force mode, and it don't have any formatting changes.

@SimonDarksideJ
Copy link
Contributor

Gah, conflicts @Cn-mjt44 , can you update from develop and resolve so we can get this merged please.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants