Skip to content

UUM-40635: Fixed issue in a number of Unity versions causing "Edit Control Scheme" menu to fail to take input - #1730

Merged
jamesmcgill merged 3 commits into
developfrom
UUM-40635/unresponsive-edit-control-scheme-menu
Aug 29, 2023
Merged

jamesmcgill merged 3 commits into
developfrom
UUM-40635/unresponsive-edit-control-scheme-menu

Conversation

@Secticide

Copy link
Copy Markdown
Contributor

Description

Issue tracker: UUM-40635

Fixed an issue causing an editor style to not be found when initialising the Edit Control Scheme menu. This PR switches from referencing the editor style via string to referencing EditorStyles.toolbarSearchField directly. This editor style is present on all versions of Unity we support and prevents us from having to go down the route of specifying which version uses which spelling.

Changes made

AdvancedDropdownGUI.Styles.toolbarSearchField now references EditorStyles.toolbarSearchField directly rather than relying on a string search.

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.
    • FogBugz ticket attached, example ([case %number%](https://issuetracker.unity3d.com/issues/...)).
    • FogBugz is marked as "Resolved" with next release version correctly set.
  • 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.

@Secticide Secticide added the bug Issues where existsing functionality misbehaves label Aug 15, 2023
@Secticide Secticide self-assigned this Aug 15, 2023
@Secticide
Secticide requested a review from ritamerkl August 15, 2023 10:53
…rectly - rather than being initialised via string
@Secticide
Secticide force-pushed the UUM-40635/unresponsive-edit-control-scheme-menu branch from 1d0fd02 to 1868acb Compare August 15, 2023 13:00
@ritamerkl
ritamerkl requested a review from Pauliusd01 August 17, 2023 07:21

@ritamerkl ritamerkl left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM, please wait for testing (@Pauliusd01)

@Pauliusd01 Pauliusd01 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM. Checked with latest trunk Editor and one version that didn't have the style name change, both work correctly:

Latest trunk (has updated style name) -> Worked before this change -> still works with this change
2021.3.27f1 (does not have updated style name) -> Did not work before this change -> now works with this change

@jamesmcgill
jamesmcgill merged commit 1fdfa3c into develop Aug 29, 2023
@jamesmcgill
jamesmcgill deleted the UUM-40635/unresponsive-edit-control-scheme-menu branch August 29, 2023 13:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Issues where existsing functionality misbehaves

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants