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: search bar shortcut block #3153

Merged
merged 4 commits into from
Jan 20, 2025
Merged

Conversation

mihakrajnc
Copy link
Contributor

What does this PR change?

This fixes the search bar disabling shortcut inputs, and cleans up same messy / incorrect gitignore.

How to test the changes?

  1. Launch the explorer
  2. Open backpack / map
  3. Click on the search bar
  4. Press Esc
  5. Shortcuts should still work afterwards

Our Code Review Standards

https://github.com/decentraland/unity-renderer/blob/master/docs/code-review-standards.md

@mihakrajnc mihakrajnc added clean-build Used to trigger clean build on PR and removed clean-build Used to trigger clean build on PR labels Jan 20, 2025
@mihakrajnc mihakrajnc enabled auto-merge (squash) January 20, 2025 21:49
Copy link

@DafGreco DafGreco left a comment

Choose a reason for hiding this comment

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

Sanity check performed in both platforms 🏁

Windows check :

  • Main Issue of the PR ✅
    This fixes the search bar disabling shortcut inputs, and cleans up same messy / incorrect gitignore.
  • Backpack (emotes, search bar & wearables) ✅
  • Camera functionality (Gallery) ✅
  • Skybox ✅
  • Map ✅
  • Notifications ✅
  • Teleport (Dollhouse / The inn / Metadynelabs) ✅

Macos sanity check :

  • Main Issue of the PR ✅
    This fixes the search bar disabling shortcut inputs, and cleans up same messy / incorrect gitignore.
  • Backpack (emotes, searchbar & wearables ) ✅
  • Emote wheel ✅
  • Teleport (Dollhouse / The inn /Metadynelabs) ✅
  • Skybox ✅
  • Map ✅
  • Camera functionality (Gallery) ✅
  • Notifications ✅

@mihakrajnc mihakrajnc merged commit 8c16219 into dev Jan 20, 2025
6 of 7 checks passed
@mihakrajnc mihakrajnc deleted the fix/search-bar-shortcut-block branch January 20, 2025 23:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clean-build Used to trigger clean build on PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[QA] Backpack-Map | Hotkeys stop working after exiting search with ESC in Backpack/Map
3 participants