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

Incorrect keys sometimes passed from Moonlight to Sunshine #963

Closed
3 tasks done
ReenigneArcher opened this issue Feb 20, 2023 · 3 comments · Fixed by #1115
Closed
3 tasks done

Incorrect keys sometimes passed from Moonlight to Sunshine #963

ReenigneArcher opened this issue Feb 20, 2023 · 3 comments · Fixed by #1115
Assignees
Labels
fixed This issue has been fixed and will be available in the next release. input:keyboard Keyboard issue os:Windows OS is Windows

Comments

@ReenigneArcher
Copy link
Member

ReenigneArcher commented Feb 20, 2023

Is there an existing issue for this?

  • I have searched the existing issues

Is your issue described in the documentation?

  • I have read the documentation

Is your issue present in the nightly release?

  • This issue is present in the nightly release

Describe the Bug

Using Moonlight Android as a client and attempting to type with the on screen keyboard will sometimes result in incorrect key presses being sent.

For example, attempting to use the capital letter "Y" will sent a lowercase "y". No capital letters are ever sent and always result in the lowercase version.

Attempting to hold the "m" button to obtain a "?" will result in a "/" character being typed on the host. Sending a "?" directly from the dedicated "?" button will also result in a "/".

Expected Behavior

No response

Additional Context

These are the incorrect symbols, anything not listed is correct or doesn't send anything.

Android Host
@ 2
# 3
$ 4
_ -
& 7
+ =
( 9
) 0
* 8
" '
: ;
! 1
? /
~ `
^ 6
{ [
} ]
% 5
< ,
> .

Additionally a | character results in \.

Symbols that don't appear on a normal keyboard, such as Pi, do not send anything (this could be due to my testing which was in Notepad).

Host Operating System

Windows

Operating System Version

11

Architecture

64 bit

Sunshine commit or version

v0.18.3

Package

Windows - installer

GPU Type

Nvidia

GPU Model

GeForce RTX 2080 Super

GPU Driver/Mesa Version

528.49

Capture Method (Linux Only)

No response

Relevant log output

No response

@cgutman
Copy link
Collaborator

cgutman commented Feb 25, 2023

This a combination of Moonlight Android issues and Sunshine issues.

The issues with capitals and symbols being entered without Shift pressed is a bug in Sunshine. It should be generating a synthetic Shift keypress if a key event comes in with the Shift modifier flag set but Shift is not already down (same with other modifiers). The Shift key is handled very strangely with most software keyboards on Android (it behaves as a toggle key, like Caps Lock), which is why it doesn't appear on other platforms or with a hardware keyboard.

The issue with special characters like Pi not working was an issue in Moonlight Android that I fixed in moonlight-stream/moonlight-android@370dbb1

@cgutman
Copy link
Collaborator

cgutman commented May 26, 2023

Fixed by 44f89de

@cgutman cgutman added the fixed This issue has been fixed and will be available in the next release. label May 26, 2023
@LizardByte-bot
Copy link
Member

This issue has been fixed and will be available in the next release.

@ReenigneArcher ReenigneArcher mentioned this issue May 29, 2023
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fixed This issue has been fixed and will be available in the next release. input:keyboard Keyboard issue os:Windows OS is Windows
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants