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: dialog window display position error #183

Merged
merged 2 commits into from
Aug 8, 2024
Merged

Conversation

myml
Copy link
Member

@myml myml commented Aug 5, 2024

日历在日程右键删除时, 弹出的确认对话框会显示在左上角
目前推测是窗管的问题, 先在应用层的临时解决

Issues: linuxdeepin/developer-center#10141

日历在日程右键删除时, 弹出的确认对话框会显示在左上角
目前推测是窗口的问题, 先在应用层临时解决

Issues: linuxdeepin/developer-center#10141
Copy link

github-actions bot commented Aug 5, 2024

TAG Bot

TAG: 5.14.3
EXISTED: no
DISTRIBUTION: unstable

发布5.14.3版本

Log: 修复确认对话框位置错误
@myml
Copy link
Member Author

myml commented Aug 5, 2024

/integrate

@deepin-ci-robot
Copy link

deepin pr auto review

关键摘要:

  • CScheduleCtrlDlg::changeEvent函数中使用QTimer::singleShot来延迟设置固定高度可能会导致用户体验不佳,因为对话框会在一个很小的延迟后出现,这可能会让人感到操作响应缓慢。
  • 代码中没有对QTimer::singleShot的回调函数进行错误处理,如果设置高度的函数执行过程中发生异常,可能会导致未处理的初始化问题。
  • 提交信息中的时间戳格式不正确,应该使用ISO 8601格式,例如YYYY-MM-DD HH:MM:SS

是否建议立即修改:

Copy link

github-actions bot commented Aug 5, 2024

AutoIntegrationPr Bot
auto integrate with pr url: deepin-community/Repository-Integration#1839
PrNumber: 1839
PrBranch: auto-integration-10244442812

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: BLumia, myml

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

@myml myml merged commit f868a6a into linuxdeepin:master Aug 8, 2024
25 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