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

全屏下运行为菜单栏应用的Motrix不能显示窗口关闭按钮 #663

Closed
KYLN24 opened this issue May 24, 2020 · 8 comments · Fixed by #667 or #670
Closed

全屏下运行为菜单栏应用的Motrix不能显示窗口关闭按钮 #663

KYLN24 opened this issue May 24, 2020 · 8 comments · Fixed by #667 or #670

Comments

@KYLN24
Copy link

KYLN24 commented May 24, 2020

错误描述
清楚简洁地描述错误,方便我们复现之后处理。
macOS 版 Motrix 1.5.10 以菜单栏应用模式打开后,进入全屏不显示窗口顶部包含“关闭、最小化”等按钮的标题栏(不知道是不是叫“标题栏”。。。)

如何重现
重现步骤,如:

  1. 设置 - 基础设置 - 运行为 - 菜单栏应用
  2. 点击窗口左上角绿色全屏按钮进入全屏
  3. 无法正确显示“标题栏”以退出全屏,只能使用 Mission Control 把全屏 Motrix 拖出来关掉

预期的行为
全屏模式运行的运行为菜单栏应用的 Motrix 应有和运行为标准应用一样的行为

运行环境

  • 操作系统类型: macOS
  • 具体版本: macOS 10.15.4
  • Motrix 版本: 1.15.10
  • 安装包类型:dmg

更多信息
补充有关该问题的其他信息。

@KYLN24
Copy link
Author

KYLN24 commented May 24, 2020

(我觉得也许不用提供截图。。。)

@agalwood
Copy link
Owner

似乎是 Electron 8.x 的 Bug,官方已经在 9.x 修复了

Motrix 下个版本才会升级到 9.x,因为需要测试和适配 Electron API 的变更

@agalwood
Copy link
Owner

用了点 hack 的方法解决了,下个小版本修复

@KYLN24
Copy link
Author

KYLN24 commented May 24, 2020

太棒了,持续跟进中~

@agalwood
Copy link
Owner

全屏模式的优化已经发布,请更新到 v1.5.13 版本

https://github.com/agalwood/Motrix/releases

@KYLN24
Copy link
Author

KYLN24 commented May 26, 2020

运行为“菜单栏应用”的 Motrix 1.5.13 在 macOS 10.15.4 上全屏运行时,点击“偏好设置”里的“保存并应用”会出现奇妙的bug(Motrix 窗口成为桌面的背景,只能通过 command + W 退出~~)

似乎是 Electron 8.x 的 Bug,官方已经在 9.x 修复了
Motrix 下个版本才会升级到 9.x,因为需要测试和适配 Electron API 的变更
用了点 hack 的方法解决了,下个小版本修复

还是等 Motrix 用上 Electron 9 好些

@KYLN24 KYLN24 reopened this May 26, 2020
@KYLN24
Copy link
Author

KYLN24 commented May 26, 2020

截屏2020-05-26 23 30 00

@agalwood
Copy link
Owner

验证了一下,只在基础设置保存时会出现,和 Electron 版本无关,是代码逻辑问题

agalwood added a commit that referenced this issue May 27, 2020
…02005271103

fix: full screen save preference issue #663
myg133 pushed a commit to myg133/Motrix that referenced this issue Jun 16, 2020
myg133 pushed a commit to myg133/Motrix that referenced this issue Jun 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants