Skip to content

FIX: Input Actions editor losing selection when the control picker opens - #2479

Merged
josepmariapujol-unity merged 6 commits into
developfrom
input/isx-2926/focus-resets
Sep 9, 2026
Merged

FIX: Input Actions editor losing selection when the control picker opens#2479
josepmariapujol-unity merged 6 commits into
developfrom
input/isx-2926/focus-resets

Conversation

@josepmariapujol-unity

@josepmariapujol-unity josepmariapujol-unity commented Sep 8, 2026

Copy link
Copy Markdown
Collaborator

Description

Backport of [UUM-150809] Fix Input Actions editor losing selection when the control picker opens #123382
https://github.cds.internal.unity3d.com/unity/unity/pull/123382

Testing status & QA

N/A

Overall Product Risks

Please rate the potential complexity and halo effect from low to high for the reviewers. Note down potential risks to specific Editor branches if any.

  • Complexity: 0
  • Halo Effect: 0

Comments to reviewers

Please describe any additional information such as what to focus on, or historical info for the reviewers.

Checklist

Before review:

  • Changelog entry added.
    • Explains the change in Changed, Fixed, Added sections.
    • For API change contains an example snippet and/or migration example.
    • JIRA ticket linked, example (case %%). If it is a private issue, just add the case ID without a link.
    • Jira port for the next release set as "Resolved".
  • Tests added/changed, if applicable.
    • Functional tests Area_CanDoX, Area_CanDoX_EvenIfYIsTheCase, Area_WhenIDoX_AndYHappens_ThisIsTheResult.
    • Performance tests.
    • Integration tests.
  • Docs for new/changed API's.
    • Xmldoc cross references are set correctly.
    • Added explanation how the API works.
    • Usage code examples added.
    • The manual is updated, if needed.

During merge:

  • Commit message for squash-merge is prefixed with one of the list:
    • NEW: ___.
    • FIX: ___.
    • DOCS: ___.
    • CHANGE: ___.
    • RELEASE: 1.1.0-preview.3.

@josepmariapujol-unity josepmariapujol-unity self-assigned this Sep 8, 2026
@josepmariapujol-unity josepmariapujol-unity changed the title [ISX-2926] Removing InputSystem icons from scene view window #122945 FIX: Removing InputSystem icons from scene view window #122945 Sep 8, 2026
@josepmariapujol-unity josepmariapujol-unity changed the title FIX: Removing InputSystem icons from scene view window #122945 FIX: Removing InputSystem icons from scene view window Sep 8, 2026
@jfreire-unity

Copy link
Copy Markdown
Collaborator

It's missing CHANGELOG.md entry btw

@u-pr u-pr Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

💡 Harness Review

Ship it

The change preserves state across project-wide action-editor rebuilds while refreshing the analytics session. I examined the rebuild lifecycle, state-copy remapping, action-change notifications, and editor teardown.

Reviewed commit c401f4b

🤖 Helpful? 👍/👎

@Pauliusd01

Pauliusd01 commented Sep 8, 2026

Copy link
Copy Markdown
Collaborator

Possibly dumb question but why is it so different from the original? Is this the right port?

@josepmariapujol-unity josepmariapujol-unity changed the title FIX: Removing InputSystem icons from scene view window FIX: Fix Input Actions editor losing selection when the control picker opens Sep 8, 2026
@josepmariapujol-unity josepmariapujol-unity changed the title FIX: Fix Input Actions editor losing selection when the control picker opens FIX: Input Actions editor losing selection when the control picker opens Sep 8, 2026
@codecov-github-com

codecov-github-com Bot commented Sep 8, 2026

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 0% with 9 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...KAssetEditor/InputActionsEditorSettingsProvider.cs 0.00% 9 Missing ⚠️
@@            Coverage Diff            @@
##           develop    #2479    +/-   ##
=========================================
  Coverage    78.95%   78.96%            
=========================================
  Files          767      767            
  Lines       140780   140914   +134     
=========================================
+ Hits        111153   111271   +118     
- Misses       29627    29643    +16     
Flag Coverage Δ
inputsystem_MacOS_6000.0 5.30% <0.00%> (-0.01%) ⬇️
inputsystem_MacOS_6000.0_project 77.50% <0.00%> (+<0.01%) ⬆️
inputsystem_MacOS_6000.3 5.30% <0.00%> (-0.01%) ⬇️
inputsystem_MacOS_6000.5 5.29% <0.00%> (-0.01%) ⬇️
inputsystem_MacOS_6000.5_project 77.55% <0.00%> (+<0.01%) ⬆️
inputsystem_MacOS_6000.6 5.29% <0.00%> (-0.01%) ⬇️
inputsystem_MacOS_6000.6_project 77.55% <0.00%> (+<0.01%) ⬆️
inputsystem_MacOS_6000.7 5.29% <0.00%> (-0.01%) ⬇️
inputsystem_MacOS_6000.7_project 77.55% <0.00%> (+<0.01%) ⬆️
inputsystem_Ubuntu_6000.0 5.31% <0.00%> (-0.01%) ⬇️
inputsystem_Ubuntu_6000.0_project 77.40% <0.00%> (-0.01%) ⬇️
inputsystem_Ubuntu_6000.3 5.31% <0.00%> (-0.01%) ⬇️
inputsystem_Ubuntu_6000.3_project 77.39% <0.00%> (-0.01%) ⬇️
inputsystem_Ubuntu_6000.5 5.30% <0.00%> (-0.01%) ⬇️
inputsystem_Ubuntu_6000.5_project 77.45% <0.00%> (-0.01%) ⬇️
inputsystem_Ubuntu_6000.6 5.30% <0.00%> (-0.01%) ⬇️
inputsystem_Ubuntu_6000.6_project 77.45% <0.00%> (-0.01%) ⬇️
inputsystem_Ubuntu_6000.7 5.30% <0.00%> (-0.01%) ⬇️
inputsystem_Ubuntu_6000.7_project 77.45% <0.00%> (-0.01%) ⬇️
inputsystem_Windows_6000.0 5.30% <0.00%> (-0.01%) ⬇️
inputsystem_Windows_6000.0_project 77.62% <0.00%> (+0.06%) ⬆️
inputsystem_Windows_6000.3 5.30% <0.00%> (-0.01%) ⬇️
inputsystem_Windows_6000.3_project 77.62% <0.00%> (+0.05%) ⬆️
inputsystem_Windows_6000.5 5.29% <0.00%> (-0.01%) ⬇️
inputsystem_Windows_6000.5_project 77.68% <0.00%> (+0.05%) ⬆️
inputsystem_Windows_6000.6 5.29% <0.00%> (-0.01%) ⬇️
inputsystem_Windows_6000.7 5.29% <0.00%> (-0.01%) ⬇️
inputsystem_Windows_6000.7_project 77.68% <0.00%> (+0.05%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...KAssetEditor/InputActionsEditorSettingsProvider.cs 0.00% <0.00%> (ø)

... and 4 files with indirect coverage changes

ℹ️ Need help interpreting these results?

Comment thread Packages/com.unity.inputsystem/CHANGELOG.md Outdated
Comment thread Packages/com.unity.inputsystem/CHANGELOG.md Outdated
@Pauliusd01
Pauliusd01 self-requested a review September 8, 2026 12:33
@josepmariapujol-unity

Copy link
Copy Markdown
Collaborator Author

Test failing unrelated to this pr

[12:52:10.206 Information] [50/1922] TestFinish
	name: APIVerificationTests.API_DocumentationManualDoesNotHaveMissingOrUnusedImages
	result: FAILED
	message: Images directory contains image files that are not referenced in any docs. Consider removing them:
  ActiveInputHandling.png
  EditorRestartWarning.png
  InstallSamples.png

@josepmariapujol-unity

Copy link
Copy Markdown
Collaborator Author

I'll fix CI in another PR

@josepmariapujol-unity
josepmariapujol-unity merged commit 6e3bb52 into develop Sep 9, 2026
93 of 107 checks passed
@josepmariapujol-unity
josepmariapujol-unity deleted the input/isx-2926/focus-resets branch September 9, 2026 08:03
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.

3 participants