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

Fix edit text button not showing and warnings on desktop #520

Merged
merged 1 commit into from
Mar 3, 2025

Conversation

Belchy06
Copy link
Collaborator

@Belchy06 Belchy06 commented Mar 3, 2025

Relevant components:

  • Signalling server
  • Common library
  • Frontend library
  • Frontend UI library
  • Matchmaker
  • Platform scripts
  • SFU

Problem statement:

Edit Text button wasn't showing on mobile. Additionally, updates to UE mean that the showOnScreenKeyboard event can be sent when on desktop leading to errors about a null editTextButton as that is only created on mobile.

Solution

Set correct display type for Edit Text button and add an additional check to check the validity of Edit Text button before trying to do anything with it.

Test Plan and Compatibility

Tested on iOS and Win11 with Editor Streaming to verify functionality and no warnings.

@Belchy06 Belchy06 added auto-backport Used to specify we want a PR to auto backport to a branch, must be paired with auto-backport-to-UEX. auto-backport-to-UE5.5 labels Mar 3, 2025
@mcottontensor mcottontensor merged commit e28eef9 into EpicGamesExt:master Mar 3, 2025
5 checks passed
Copy link
Contributor

github-actions bot commented Mar 3, 2025

💚 All backports created successfully

Status Branch Result
UE5.5

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

mcottontensor added a commit that referenced this pull request Mar 3, 2025
[UE5.5] Merge pull request #520 from Belchy06/temp
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Used to specify we want a PR to auto backport to a branch, must be paired with auto-backport-to-UEX. auto-backport-to-UE5.5
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants