Skip to content

Conversation

@yixinshark
Copy link
Contributor

lock widget need update Spacer item size when lock widget was set visible by DBus interface

Log: as title
Pms: BUG-313339

lock widget need update Spacer item size when lock widget  was set visible by DBus interface

Log: as title
Pms: BUG-313339
@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: yixinshark

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

Details 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

deepin-ci-robot added a commit to linuxdeepin/dde-session-shell-snipe that referenced this pull request Jun 9, 2025
Synchronize source files from linuxdeepin/dde-session-shell.

Source-pull-request: linuxdeepin/dde-session-shell#12
@deepin-ci-robot
Copy link

deepin pr auto review

代码审查意见:

  1. .cursorindexingignore.specstory/.gitignore文件中,注释应该使用统一的格式,例如使用中文或英文,避免混合使用。
  2. .specstory/.gitignore文件中的注释应该更详细地说明.what-is-this.md文件的作用。
  3. updateBlurEffectGeometry方法中,使用了qMin函数来计算高度,但是没有考虑到不同屏幕分辨率和UI缩放的情况。建议使用更稳定的计算方法,例如基于窗口高度的比例。
  4. updateBlurEffectGeometry方法中,添加了日志输出,这对于调试是有帮助的,但是应该避免在生产环境中留下这些日志输出。
  5. updateBlurEffectGeometry方法中,使用了std::min函数来计算高度,但是没有包含<algorithm>头文件。应该添加#include <algorithm>来确保编译通过。
  6. SFAWidget::showEvent方法中,添加了updateSpaceItemupdateBlurEffectGeometry的调用,但是没有提供这些方法的实现。应该确保这些方法已经实现,并且不会引入新的问题。
  7. SFAWidget::showEvent方法中,使用了QTimer::singleShot来延迟调用updateBlurEffectGeometry方法,但是没有提供延迟时间。建议提供一个合理的延迟时间,以确保布局已经完成。
  8. SFAWidget::showEvent方法中,使用了QTimer::singleShot来延迟调用updateFocus方法,但是没有提供延迟时间。建议提供一个合理的延迟时间,以确保焦点已经正确设置。

总的来说,代码的修改应该保持原有的布局逻辑,同时添加适当的检测和修复机制,以确保在不同情况下都能正确计算模糊背景的高度。同时,应该注意代码的可读性和可维护性,避免在生产环境中留下调试用的日志输出。

@yixinshark yixinshark closed this Jun 9, 2025
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.

2 participants