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.

Sign up for GitHub

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

Jump to bottom

Publisher: Ignore escape button #4090

Merged
merged 1 commit into from Nov 11, 2022
Merged

Publisher: Ignore escape button #4090

merged 1 commit into from Nov 11, 2022
+7 −0

Conversation

iLLiCiTiT
Copy link
Member

Brief description

Don't close publisher window on press of escape button.

Testing notes:

  1. Open publisher in host or open tray publisher
  2. Hit escape on keyboard
  3. Nothing should happen

@iLLiCiTiT iLLiCiTiT self-assigned this Nov 11, 2022
@iLLiCiTiT iLLiCiTiT added the type: enhancement Enhancements to existing functionality label Nov 11, 2022
@iLLiCiTiT iLLiCiTiT merged commit 5380648 into develop Nov 11, 2022
@iLLiCiTiT iLLiCiTiT deleted the feature/publisher_ignores_esc_btn branch November 11, 2022 11:58
@github-actions github-actions bot added this to the next-patch milestone Nov 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@64qam 64qam 64qam approved these changes

@jakubjezek001 jakubjezek001 Awaiting requested review from jakubjezek001

@kalisp kalisp Awaiting requested review from kalisp

Assignees

@iLLiCiTiT iLLiCiTiT

Labels
type: enhancement Enhancements to existing functionality
Projects
None yet
Milestone
3.14.7
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@iLLiCiTiT @64qam