Add dispatcher for keyboard shortcuts
References #1978.
Pull Request checklist
- [ ] Quality: This PR builds and passes detekt/ktlint checks (A pre-push hook is recommended)
- [ ] Tests: This PR includes thorough tests or an explanation of why it does not
- [ ] Changelog: This PR includes a changelog entry or does not need one
- [ ] Accessibility: The code in this PR follows accessibility best practices or does not include any user facing features
After merge
- Milestone: Make sure issues closed by this pull request are added to the milestone of the version currently in development.
- Breaking Changes: If this is a breaking change, please push a draft PR on Reference Browser to address the breaking issues.
This is more of a proof-of-concept to show interest and ask for help. @samlh @pocmo could you give me guidance on how this might be implemented in a more extensible way? I understand that this should be sending these events to the embedding application so it can call things like components.useCases.sessionUseCases.reload() but I'm not sure where to start.
Codecov Report
Merging #9948 (97b802f) into master (055736c) will decrease coverage by
8.37%. The diff coverage isn/a.
@@ Coverage Diff @@
## master #9948 +/- ##
============================================
- Coverage 74.00% 65.63% -8.38%
+ Complexity 5665 269 -5396
============================================
Files 778 94 -684
Lines 28831 2258 -26573
Branches 4741 363 -4378
============================================
- Hits 21337 1482 -19855
+ Misses 5013 618 -4395
+ Partials 2481 158 -2323
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing dataPowered by Codecov. Last update 055736c...97b802f. Read the comment docs.
This pull request has conflicts when rebasing. Could you fix it @gaul? 🙏
This pull request has conflicts when rebasing. Could you fix it @gaul? 🙏