Skip to content

Conversation

@18202781743
Copy link
Contributor

  • fix: 配合Qt删除_d_dtk_lineedit**属性
  • fix: 修复combobox下拉按钮宽高不等, 还有按钮与编辑框不贴合的问题
  • fix: 进度条圆角问题以及当进度为1时进度滑块未在进度槽中的问题
  • fix: 修复搜索和combobox下拉框动效问题
  • fix: 修复DTabBar边界和悬浮样式
  • fix: 修复窗口菜单和窗口内容混为一体
  • fix: 修复主窗口多次切换主题时偶发性崩溃
  • fix: 调整按钮圆角,区分普通状态和在编辑框中的状态
  • fix: 优化判断是否为combobox内按钮的方式
  • fix: 修复live系统恢复页面文管对话框盘符显示不完整
  • fix: 修复【notepadqq】顶部工具栏点击设置-首选项应用崩溃
  • chore: 重绘滑动条滑块的焦点效果
  • fix: 密码框与显示密码按钮高度不一致
  • fix: 重启任务栏右键菜单概率不显示文字内容

Whale107 and others added 14 commits October 17, 2025 14:25
配合Qt删除_d_dtk_lineedit**属性

Log: 配合Qt删除_d_dtk_lineedit**属性
Bug: https://pms.uniontech.com/bug-view-200405.html
     https://pms.uniontech.com/bug-view-231425.html
Influence: 使用DTK DLineEdit的应用
Change-Id: Icd891fe33dbaa89ba2410fc16a7cf710cdd410c5
在矩形还没做正方形变换前就对rect进行了移动,导致移动的距离是(编辑框宽度-原本按钮宽度)
正确的移动距离应是(编辑框宽度-正方形变换后的按钮宽度)
调整了编辑框按钮的圆角和边距,保证和编辑框贴合

Log: 修复combobox下拉按钮长宽不等, 还有按钮与编辑框不贴合的问题
Bug: https://pms.uniontech.com/bug-view-247173.html
     https://pms.uniontech.com/bug-view-184543.html
Influence: 所有可编辑combobox下拉按钮
Change-Id: I97f352e6c70201f3e6b0b19d24a843da4ebd0e07
进度条圆角问题以及当进度为1时进度滑块未在进度槽中的问题

Log: 进度条圆角问题以及当进度为1时进度滑块未在进度槽中的问题
Bug: https://pms.uniontech.com/bug-view-230475.html
Influence: DTK进度条
Change-Id: Ie14f0bdbbee7cd915765aceefd529d411121dacb
修复搜索和combobox下拉框动效问题

Log: 修复搜索和combobox下拉框动效问题
Bug: https://pms.uniontech.com/bug-view-243363.html
Influence: 搜索和ComBobox下拉框动效
Change-Id: Ic886448c353a3ea6d6fa9619f9e9c5c89ebd6bad
去除边界,修改悬浮样式

Log: 修复DTabBar边界和悬浮样式
Bug: https://pms.uniontech.com/bug-view-249343.html
Influence: 终端,记事本标签
Change-Id: I287316e02c692dabd59cc8e5aae3cff14f88e6a9
使用窗管接口给窗口菜单添加边界

Log: 修复窗口菜单和窗口内容混为一体
Bug: https://pms.uniontech.com/bug-view-243155.html
Influence: 主窗口菜单
Change-Id: Ie948289312919f14c9b17801beb9f6b0137cac9a
主窗口和主窗口内的窗口都监听了主题变化的信号,并且调用相同了接口,
在菜单的连接中信号的接收者有误,this的话就是主窗口了,会产生冲突

Log: 复主窗口多次切换主题时偶发性崩溃
Task: https://pms.uniontech.com/task-view-337571.html
Influence: 窗口主题
Change-Id: If8050a8844bd975ab24dd791cccb5e14a6efd95c
绘制按钮时判断按钮是否在combobox中, 是则圆角减小1像素, 否则不做处理

Log: 调整按钮圆角,区分普通状态和在编辑框中的状态
Bug: https://pms.uniontech.com/bug-view-250563.html
Influence: DTK按钮圆角
Change-Id: I725aea650eee17095a122ff2b599e951c8f6e8d5
不使用判断AssesibleName的方式,判断类型是否为QDateTimeEdit并且开启了calendarPopup,
 则为编辑框内按钮,和combobox做相同的圆角处理
Log: 优化判断是否为combobox内按钮的方式
Bug: https://pms.uniontech.com/bug-view-250563.html
Influence: DTK按钮圆角

Change-Id: Ib695272d69b4f3c60929f8e12b2820988aee10c2
绘制进度条2设置了裁切,绘制完后应关闭裁切,避免影响后续的绘制

Log: 修复live系统恢复页面文管对话框盘符显示不完整
Bug: https://pms.uniontech.com/bug-view-252645.html
Influence: 进度条 文管对话框
Change-Id: I56ce6084a75a9a682db97745cc10504ff73f314d
在使用styleObject对象的成员时未对对象做判空操作

Log: 修复【notepadqq】顶部工具栏点击设置-首选项应用崩溃
Bug: https://pms.uniontech.com/bug-view-251999.html
Influence: notepadqq
Change-Id: I58479e7b0a3e267694272036c7bec9e08c4cff5a
重绘滑动条滑块的焦点效果

Log: 重绘滑动条滑块的焦点效果
Bug: https://pms.uniontech.com/bug-view-256291.html
Influence: 所有滑动条的焦点效果
Change-Id: I1aa4a7cdb75e1a8ccae52500c92e84f60b6e405f
绘制按钮矩形不能减去边界,否则会比输入框矮两个像素(上下各1px)

Log: 密码框与显示密码按钮高度不一致
Bug: https://pms.uniontech.com/bug-view-250563.html
Influence: 按钮大小
Change-Id: Ia004950b06663982b0ca2f1e92611ba35f2ede03
原因是QFontInfo中的指针字体大小为无穷大,导致设置字体无法显示出来
而QFont里面字体大小是正常的,需要Qt具体分析根因

Log: 重启任务栏概率右键菜单不显示文字内容
Influence: 任务栏邮件菜单显示
Bug: CPM20250312001539
Change-Id: I963b9f2bdcb576bd1bea38b43a3a778e3d687283
@18202781743 18202781743 requested review from BLumia and mhduiy October 17, 2025 07:33
@deepin-ci-robot
Copy link
Contributor

deepin pr auto review

我来对这个diff进行代码审查,主要关注语法逻辑、代码质量、性能和安全性。

  1. 语法逻辑改进:
  • 在drawSliderHandleFocus函数中,将重复的路径绘制代码重构为匿名函数createPath,提高了代码复用性和可维护性。
  • 在CE_ProgressBarGroove和CE_ProgressBarContents的处理中,简化了frameRadius的计算逻辑,直接使用height/2,避免了不必要的条件判断。
  1. 代码质量改进:
  • 添加了更多的空行分隔不同的代码块,提高了代码可读性。
  • 统一了代码风格,如变量命名和注释格式。
  • 在drawControl函数中,对按钮状态的判断逻辑更加清晰,使用了else if结构避免了嵌套过深。
  1. 性能改进:
  • 在QSlider的处理中,优化了绘制路径的计算方式,减少了重复计算。
  • 在setWindowNoEffect函数中添加了版本检查,避免在不支持的版本上执行不必要的操作。
  1. 安全性改进:
  • 在访问styleObject属性前添加了空指针检查,防止潜在的空指针访问:
if (opt->styleObject && opt->styleObject->property("_d_slider_spaceLeft").isValid())
  • 在设置窗口边框颜色时,添加了主题类型检查,确保颜色值的有效性。
  1. 具体修改建议:
  • 在drawSliderHandleFocus函数中,可以将getThemTypeColor函数的定义移到更合适的位置,避免重复调用。
  • 在polish函数中,设置边框颜色的代码可以提取为单独的函数,提高代码复用性。
  • 在QComboBox的下拉按钮处理中,建议添加对buttonRectSize的最小值限制,避免按钮过小影响用户体验。
  1. 其他建议:
  • 建议在drawPrimitive函数中添加对frame_radius的边界检查,防止负值或过大值导致的绘制异常。
  • 在setBorderColor函数中,建议添加对color参数的有效性检查。
  • 建议为关键的绘制操作添加性能日志,便于性能分析和优化。

总体来说,这个diff的修改质量较高,主要改进了UI绘制的细节和代码结构,但还需要注意一些边界情况的处理和性能优化。

deepin-ci-robot added a commit to linuxdeepin/qt6integration that referenced this pull request Oct 17, 2025
Synchronize source files from linuxdeepin/qt5integration.

Source-pull-request: linuxdeepin/qt5integration#286
@deepin-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: 18202781743, BLumia, mhduiy

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

@18202781743
Copy link
Contributor Author

/topic dtk-cpv20

@deepin-ci-robot
Copy link
Contributor

Add topic: dtk-cpv20 successed.

@18202781743 18202781743 merged commit 4e1d7b9 into linuxdeepin:master Nov 6, 2025
21 of 22 checks passed
@github-project-automation github-project-automation bot moved this to Done in dtk-cpv20 Nov 6, 2025
18202781743 pushed a commit to linuxdeepin/qt6integration that referenced this pull request Nov 6, 2025
Synchronize source files from linuxdeepin/qt5integration.

Source-pull-request: linuxdeepin/qt5integration#286
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

6 participants