Skip to content

DOCF-3045 Clarified information about key layout, text input and key controls#1695

Merged
lyndon-unity merged 10 commits intodevelopfrom
DOCF-3045-keycodes-text-input-mapping-in-JIS
Jul 27, 2023
Merged

DOCF-3045 Clarified information about key layout, text input and key controls#1695
lyndon-unity merged 10 commits intodevelopfrom
DOCF-3045-keycodes-text-input-mapping-in-JIS

Conversation

@duckets
Copy link
Copy Markdown
Contributor

@duckets duckets commented Jun 28, 2023

Description

https://jira.unity3d.com/browse/DOCF-3045
Some API documentation did not clearly distinguish between key input controls and keyboard text input, particularly in relation to variations in keyboard layout due to system language. This was causing confusion for users who were (incorrectly) trying to read text input using keycodes.

Changes made

Added documentation to clarify usage of KeyControl and Keyboard.Key, and added links to Keyboard.onTextInput.

@duckets duckets requested a review from lyndon-unity June 28, 2023 13:04
Copy link
Copy Markdown
Collaborator

@lyndon-unity lyndon-unity left a comment

Choose a reason for hiding this comment

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

Changes look reasonable. Happy to approve.

However these is a test failure with the documentation
Looks like the CapsLock link and the Controls.KeyControl link aren't valid

playmode Test Failure: DocumentationBasedAPIVerficationTests.API_DocsDoNotHaveXMLDocErrors
Expected:
But was: < "[23-06-28 01:22:56.342]Warning:[MetadataCommand.ExtractMetadata]Invalid cref value "!:A" found in triple-slash-comments for KeyControl defined in Packages/com.unity.inputsystem/InputSystem/Controls/KeyControl.cs Line 36, ignored.", "[23-06-28 01:22:56.343]Warning:[MetadataCommand.ExtractMetadata]Invalid cref value "!:CapsLock" found in triple-slash-comments for KeyControl defined in Packages/com.unity.inputsystem/InputSystem/Controls/KeyControl.cs Line 36, ignored.", "[23-06-28 01:22:56.343]Warning:[MetadataCommand.ExtractMetadata]Invalid cref value "!:see Controls.KeyControl" found in triple-slash-comments for keyCode defined in Packages/com.unity.inputsystem/InputSystem/Controls/KeyControl.cs Line 46, ignored." >

@duckets duckets marked this pull request as draft July 4, 2023 08:58
@duckets duckets marked this pull request as ready for review July 17, 2023 08:27
Copy link
Copy Markdown
Collaborator

@lyndon-unity lyndon-unity left a comment

Choose a reason for hiding this comment

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

Looks like a clean updated, most CI passing, one unrelated failure, I've just triggered that again to be sure.

@lyndon-unity lyndon-unity merged commit 37ee762 into develop Jul 27, 2023
@lyndon-unity lyndon-unity deleted the DOCF-3045-keycodes-text-input-mapping-in-JIS branch July 27, 2023 10:35
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