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

feat: 重构按键输入处理,并支持部分快捷键 #321

Closed
wants to merge 2 commits into from

Conversation

WhiteXero
Copy link
Contributor

重构应用对各种按键输入的处理,并统一由快捷键处理器管理

受影响的键位:

  • 全局
    • Shift+H 最小化应用(老板键)
  • 视频播放页
    • ↑ ↓ 音量控制
    • ← → 进度控制
    • Space 暂停或继续播放
    • Esc 退出全屏
    • D 弹幕控制
    • F 全屏控制

各个作用域的快捷键将在加载时进行注册

#299 Q2及未来其他快捷键实现的初步准备

Closes #264

@WhiteXero
Copy link
Contributor Author

WhiteXero commented Oct 18, 2024

failed: Linux 需要 KeyBinder-3.0 库以实现功能

@Predidit
Copy link
Owner

@WhiteXero 我们是否可以将这个PR暂且标记为草稿,因为在我看来有以下三个主要问题

  1. 我们没有在 player_item 析构时注销这些热键,这些按键在视频播放页面之外,以及在视频播放页内更换选集导致播放器重新加载时的行为不明确。

  2. 热键捕获导致了对应按键在弹幕检索时失效,我们无法使用这些热键拼写番剧名。

  3. 增加自动暂停和播放源检测;修复macOS和iOS的bug #317 合并之后,此PR存在冲突需要处理.

@WhiteXero
Copy link
Contributor Author

当然,稍后将修复这些问题

@Predidit
Copy link
Owner

此外我认为 leanflutter/hotkey_manager#12 也是我们暂时无法解决的问题。

实际上或许使用flutter原生组件和焦点机制捕获键盘事件没有什么不妥?

@WhiteXero WhiteXero marked this pull request as draft October 19, 2024 03:23
@WhiteXero WhiteXero closed this Nov 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

能否为Windows端添加键盘上下方向键控制音量的功能
2 participants