Skip to content

fix: 修复SwitchButton在透明窗口上显示时颜色太亮,太突兀的问题#651

Merged
deepin-bot[bot] merged 1 commit intolinuxdeepin:masterfrom
18202781743:master
Apr 8, 2025
Merged

fix: 修复SwitchButton在透明窗口上显示时颜色太亮,太突兀的问题#651
deepin-bot[bot] merged 1 commit intolinuxdeepin:masterfrom
18202781743:master

Conversation

@18202781743
Copy link
Contributor

原本使用的色板中button的颜色,现改为带透明度的黑色和白色

Log: 修复SwitchButton在透明窗口上显示时颜色太亮,太突兀的问题
Bug: https://pms.uniontech.com/bug-view-250757.html
Influence: DTK SwitchButton按钮 (开关)
Change-Id: Ifb231a3a9ec548d6970d240eaf2667e50ffed9f3

原本使用的色板中button的颜色,现改为带透明度的黑色和白色

Log: 修复SwitchButton在透明窗口上显示时颜色太亮,太突兀的问题
Bug: https://pms.uniontech.com/bug-view-250757.html
Influence: DTK SwitchButton按钮 (开关)
Change-Id: Ifb231a3a9ec548d6970d240eaf2667e50ffed9f3
@18202781743 18202781743 requested review from BLumia and mhduiy April 8, 2025 06:42
@deepin-ci-robot
Copy link
Contributor

deepin pr auto review

关键摘要:

  • 代码中使用了硬编码的颜色值,建议使用常量或配置文件来管理颜色值,以便于维护和修改。
  • setAlphaF 方法用于设置颜色的透明度,但未对透明度值进行验证,可能会导致超出预期范围的透明度。

是否建议立即修改:

deepin-ci-robot added a commit to linuxdeepin/dtk6widget that referenced this pull request Apr 8, 2025
Synchronize source files from linuxdeepin/dtkwidget.

Source-pull-request: linuxdeepin/dtkwidget#651
@18202781743
Copy link
Contributor Author

@deepin-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: 18202781743, 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

/forcemerge

18202781743 pushed a commit to linuxdeepin/dtk6widget that referenced this pull request Apr 8, 2025
Synchronize source files from linuxdeepin/dtkwidget.

Source-pull-request: linuxdeepin/dtkwidget#651
@deepin-bot
Copy link
Contributor

deepin-bot bot commented Apr 8, 2025

This pr force merged! (status: blocked)

@deepin-bot deepin-bot bot merged commit 4fc0362 into linuxdeepin:master Apr 8, 2025
16 of 21 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.

4 participants