Skip to content

Conversation

@Darshan-upadhyay1110
Copy link
Contributor

@Darshan-upadhyay1110 Darshan-upadhyay1110 commented Jan 5, 2026

Backport: #13944

  • Before this patch keybindings were not working on start slide show presentation
  • we do added event listener with capture phase so the keydown will not get override by other key binding
  • start window needs to have the the top priority for arrow key listeners

This fixes an issue in Firefox where arrow keys would not reliably trigger slide changes when presenting in window mode.

Change-Id: I604c1df0ace918d4bda4877390e92837b9b058d9

  • Resolves: #
  • Target version: main

Summary

TODO

  • ...

Checklist

  • I have run make prettier-write and formatted the code.
  • All commits have Change-Id
  • I have run tests with make check
  • I have issued make run and manually verified that everything looks okay
  • Documentation (manuals or wiki) has been updated or is not required

- Before this patch keybindings were not working on start slide show presentation
- we do added event listener with capture phase so the `keydown` will not get override by  other key binding
- start window needs to have the the top priority for arrow key listeners

This fixes an issue in Firefox where arrow keys would not reliably
trigger slide changes when presenting in window mode.

Signed-off-by: Darshan-upadhyay1110 <[email protected]>
Change-Id: I604c1df0ace918d4bda4877390e92837b9b058d9
@github-project-automation github-project-automation bot moved this from To Review to To Test in Collabora Online Jan 6, 2026
@eszkadev eszkadev merged commit 7eeea59 into distro/collabora/co-25.04 Jan 6, 2026
3 checks passed
@eszkadev eszkadev deleted the private/darshan/backport-13944 branch January 6, 2026 09:52
@github-project-automation github-project-automation bot moved this from To Test to Done in Collabora Online Jan 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants