chore: animation tweak - Githubissues

linuxdeepin / dtkdeclarative

Other

20 stars 40 forks source link

chore: animation tweak #384

Closed kegechen closed 2 months ago

kegechen commented 2 months ago

switch animation duration 500ms
listview hovered background color remove shadowcolor
listview hovered background size adapt to hovered item's background

deepin-ci-robot commented 2 months ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: kegechen

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/dev/animation/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-bot[bot] commented 2 months ago

Doc Check bot :green_circle: Document Coverage Check Passed!

deepin-ci-robot commented 2 months ago

deepin pr auto review

关键摘要:

在ListView的Item中，implicitWidth和implicitHeight的计算逻辑被修改，可能会影响列表项的布局和显示。需要确认这是否是预期的更改。
在Switch组件中，toggletimer的interval被修改为500毫秒，这可能会影响切换操作的响应性。需要评估这是否是合适的更新。

是否建议立即修改:

是，需要确保这些更改是经过充分测试和评估的，以避免潜在的布局和功能问题。