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

fix: 修复大窗口关闭看图后,改变增加缩放比后,打开看图无法看到看图标题栏问题 #208

Merged
merged 1 commit into from
Jan 8, 2025

Conversation

myk1343
Copy link

@myk1343 myk1343 commented Jan 8, 2025

修复大窗口关闭看图后,改变增加缩放比后,打开看图无法看到看图标题栏问题

Bug: https://pms.uniontech.com/bug-view-298659.html
Log: 修复大窗口关闭看图后,改变增加缩放比后,打开看图无法看到看图标题栏问题

修复大窗口关闭看图后,改变增加缩放比后,打开看图无法看到看图标题栏问题

Bug: https://pms.uniontech.com/bug-view-298659.html
Log: 修复大窗口关闭看图后,改变增加缩放比后,打开看图无法看到看图标题栏问题
@deepin-ci-robot
Copy link

deepin pr auto review

关键摘要:

  • 在新增代码中,对窗口位置进行了检查和调整,确保窗口不会出现在屏幕之外。

是否建议立即修改:

  • 否,当前提交的代码逻辑清晰,没有明显的语法或逻辑错误。
  • 是,建议添加注释说明为什么需要检查窗口位置,以及如何处理窗口位置不正确的情况。
  • 是,建议在调整窗口位置时,考虑窗口的最小尺寸,避免窗口太小影响用户体验。

其他建议:

  • 确保在调整窗口位置时,不会覆盖其他应用程序或系统托盘等关键区域。
  • 考虑使用更优雅的方法来处理窗口位置,例如使用QDesktopWidget来获取屏幕尺寸和可用空间。

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: myk1343, pengfeixx

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@myk1343
Copy link
Author

myk1343 commented Jan 8, 2025

/merge

@deepin-bot deepin-bot bot merged commit d6a1b10 into linuxdeepin:release/eagle Jan 8, 2025
15 checks passed
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.

3 participants