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

通过快捷键将倍速从2倍升至2.5倍后，无法通过快捷键或点击倍速按钮，将倍速改回2倍速 #1504

Closed
youngerstar opened this issue Jan 29, 2021 · 4 comments · Fixed by #1567
Closed

通过快捷键将倍速从2倍升至2.5倍后，无法通过快捷键或点击倍速按钮，将倍速改回2倍速 #1504

youngerstar opened this issue Jan 29, 2021 · 4 comments · Fixed by #1567

Comments

@youngerstar
Copy link

youngerstar commented Jan 29, 2021 •
edited
Loading

No description provided.

@youngerstar youngerstar changed the title 全屏状态下，通过快捷键将倍速升至3 全屏状态下，通过快捷键将倍速从2倍升至2.5倍后，无法通过快捷键或点击倍速按钮，将倍速改回2倍速 Jan 29, 2021
@youngerstar youngerstar changed the title 全屏状态下，通过快捷键将倍速从2倍升至2.5倍后，无法通过快捷键或点击倍速按钮，将倍速改回2倍速 通过快捷键将倍速从2倍升至2.5倍后，无法通过快捷键或点击倍速按钮，将倍速改回2倍速 Jan 29, 2021
@LonelySteve
Copy link
Contributor

LonelySteve commented Jan 29, 2021 •
edited
Loading

你好，请问浏览器控制台有输出什么错误信息吗？

另外，请尝试使用最新预览版，由于预览版和正式版的配置是独立的，因此如果在导入你现有的配置文件之后仍能复现此问题，请提供你的配置文件，如果该问题仅发生在特定视频页，还请提供该视频页的地址。

LonelySteve added a commit to LonelySteve/Bilibili-Evolved that referenced this issue Jan 30, 2021
@LonelySteve
fix(video-speed-controller): fix the1812#1504
5de960c 
* API 变化
  - `getInstance` 和 `constructor` 均新增一个参数，用于传递原生倍速值
  - 在 `containerElement` 元素上分发的自定义事件的 `detail` 属性新增一个 `isNativeSpeed` 属性，用于表示变更的倍速是否为原生倍速
* 功能变化
  - 现在缓存命中机制除了考虑 `containerElement` 元素，还会比较上一次倍数值和原生倍速值
  - 修正了内部原生倍速值的获取方法，现在优先使用传入的原生倍速值，其次尝试使用上一次倍数值，最后才使用默认的 1.0x 值
@LonelySteve
Copy link
Contributor

@youngerstar 我已经在 #1509 中修复了此问题，不过理论上来说，正式版应该没有这个问题，这个 BUG 应该是在 v1.11.12 之后引入的。

你使用 v1.11.12 的版本，也是在换 P 的时候出现这个问题吗？

@LonelySteve
Copy link
Contributor

LonelySteve commented Jan 30, 2021 •
edited
Loading

没能在正式版中复现此问题，你可以先导出保存一下自己的设置，然后重装一下最新的正式版，看问题是否解决（注意，这个版本有个 BUG，扩展倍速要打开「记忆上次播放速度」功能才能用）。

image

如果问题解决，说明可能是你的设置项导致了这个问题，请提供你的设置文件以供后续测试。

如果问题依旧，那就先等待预览版更新吧。

the1812 added a commit that referenced this issue Feb 1, 2021
@the1812
Merge pull request #1509 from LonelySteve/fix/#1504
55764b7 
fix #1504
@LonelySteve
Copy link
Contributor

@youngerstar 新的预览版已经发布，能帮忙确认一下问题是否已经解决了吗？

@the1812 the1812 mentioned this issue Feb 17, 2021
Merged
LonelySteve added a commit to LonelySteve/Bilibili-Evolved that referenced this issue Feb 19, 2021
@LonelySteve
fix(video speed): fix the1812#1504 again
40820c7
@LonelySteve LonelySteve mentioned this issue Feb 19, 2021
Merged
the1812 pushed a commit that referenced this issue Mar 2, 2021
@LonelySteve @the1812
fix(video speed): fix #1504 again
3f9920e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update to v1.12.0 the1812/Bilibili-Evolved
2 participants
@LonelySteve @youngerstar