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

为 macOS 添加了专用窗口属性 #65

Merged
merged 9 commits into from
Jan 23, 2025

Conversation

CodeCrafter-TL
Copy link
Contributor

@CodeCrafter-TL CodeCrafter-TL commented Jan 22, 2025

PR Summary

为 macOS 平台添加了专用的窗口属性:

  • -modified: 是否使窗口显示为“已修改”状态
  • -transparent: 是否使窗口背景透明

Extra Information

PR Checklist

  • The PR doesn't duplicate another PR which is already open
  • New or changed code has been tested
  • When it comes to API modifications, the docstrings of related classes or functions are also modified
  • I have read the contribution guide and followed all the instructions

Copy link

codecov bot commented Jan 22, 2025

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

ℹ️ You can also turn on project coverage checks and project coverage reporting on Pull Request comment

Thanks for integrating Codecov - We've got you covered ☂️

@Xiaokang2022 Xiaokang2022 self-assigned this Jan 22, 2025
@Xiaokang2022 Xiaokang2022 added the ✨ Enhancement New feature or request label Jan 22, 2025
Copy link
Owner

@Xiaokang2022 Xiaokang2022 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

代码中还有一些空行存在空格字符,这些是不需要的

maliang/core/containers.py Outdated Show resolved Hide resolved
maliang/core/containers.py Outdated Show resolved Hide resolved
@Xiaokang2022
Copy link
Owner

另外,docstring 存在一些语法错误,也改一下

@CodeCrafter-TL
Copy link
Contributor Author

我已修改

@CodeCrafter-TL
Copy link
Contributor Author

我已修改

Copy link
Owner

@Xiaokang2022 Xiaokang2022 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@Xiaokang2022 Xiaokang2022 merged commit b316d7c into Xiaokang2022:main Jan 23, 2025
16 checks passed
@Xiaokang2022
Copy link
Owner

非常感谢 @CodeCrafter-TL 的贡献!❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants