fix: theme type can't change for notification

[18202781743]

as title.

pms: Bug-283995,Bug-283991

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

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

• 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

[deepin-ci-robot]

deepin pr auto review

代码审查意见：

1. 模块导入的版本号被移除：

   • 在main.qml文件中，模块导入的版本号从2.15和1.15被移除。虽然这在某些情况下是允许的，但建议保留版本号以确保兼容性。如果确定不需要特定版本，可以保留版本号以避免未来可能出现的兼容性问题。

2. palette属性的使用：

   • 在两个main.qml文件中，都添加了palette: DTK.palette属性。这可能会影响窗口的外观，但需要确认DTK.palette是否已经正确初始化，并且是否适用于所有情况。如果DTK.palette的值可能会变化，建议添加适当的检查或默认值。

3. 窗口边距的调整：

   • 在notification/center/package/main.qml文件中，窗口的边距被增加了10。这可能会导致窗口与屏幕边缘的距离变大，影响用户体验。建议确认这一改动是否符合设计要求，并考虑是否有必要提供配置选项来调整这些值。

4. 代码风格一致性：

   • 虽然代码改动不大，但建议检查整个项目中是否有统一的代码风格，包括导入语句的格式、缩进等。不一致的代码风格可能会影响代码的可读性和维护性。

5. 缺少注释：

   • 新增的palette属性和窗口边距调整的改动没有相应的注释说明。虽然这些改动可能是必要的，但添加注释可以帮助其他开发者理解这些改动的目的和影响。

总体来说，这些改动看起来是合理的，但需要确保它们不会引入任何问题，并且与项目的整体设计和目标保持一致。

[18202781743]

/forcemerge

[deepin-bot]

This pr force merged! (status: blocked)