fix: search icon not visible

asterwyx commented 6 months ago

Set icon source height explicitly.

Log: fix search icon not visible Issue: https://github.com/linuxdeepin/developer-center/issues/8306

deepin-bot[bot] commented 6 months ago

Doc Check bot :green_circle: Document Coverage Check Passed!

deepin-ci-robot commented 6 months ago

deepin pr auto review

关键摘要:

sourceSize 属性改为直接使用 Qt.size 函数可能意味着意图从响应式设计 systems 切换到直接指定大小。确认这是否符合设计要求。
检查是否有必要在所有使用 sourceSize 的地方调用 Qt.size，以确保一致性和可预测性。

合并相似的问题是否建议立即修改:

是，需要确认 sourceSize 属性修改的意图，并确保这种改动不会影响现有的布局和样式。如果这是一个故意的调整，需要确保所有相关的部分都已经更新以反映这一变化。如果这是一个无意的错误，那么应该恢复原来的代码以避免可能的布局问题。

deepin-ci-robot commented 6 months ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: 18202781743, asterwyx

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

Needs approval from an approver in each of these files: - **[debian/deepin/OWNERS](https://github.com/linuxdeepin/dtkdeclarative/blob/master/debian/deepin/OWNERS)** Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment

linuxdeepin / dtkdeclarative

fix: search icon not visible #336

deepin pr auto review