Winlayout restore on output changes

ZhongYic00 commented 5 months ago

Impl basic winlayout restore on output changes.

outputs environment
- use the set of outputs as hash-key
state restore
- only restore the surfaces that has a matched state
- use intermediate state for transition animating

deepin-ci-robot commented 5 months ago

Skipping CI for Draft Pull Request. If you want CI signal for your change, please convert it to an actual PR. You can still manually trigger a test run with /test all

deepin-ci-robot commented 5 months ago

deepin pr auto review

关键摘要:

在QmlHelper.qml文件中，outputSet的定义应该在DynamicCreator组件内部，而不是作为属性直接在Item组件中。
updateEnvHash函数中的console.log调用可能会导致性能问题，应该移除或替换为更高效的数据处理方式。
onEnvHashChanged事件处理函数中的console.log调用也应该被移除或替换，以避免潜在的性能问题。
SurfaceWrapper.qml文件中的move函数中的isMoveResizing和manualMoveResizing变量应该在函数顶部声明，而不是在每次调用move函数时重新赋值。
SurfaceWrapper.qml文件中的saveState和restoreState函数中使用了console.debug，这可能不是生产环境所需要的，应该移除或替换为更合适的日志记录方式。
SurfaceWrapper.qml文件中的store变量应该有一个更具体的类型定义，以便更好地表示其用途。
SurfaceWrapper.qml文件中的onIsMoveResizingChanged事件处理函数中，当state为空时调用saveState可能会导致未定义行为，应该添加相应的检查或处理逻辑。
SurfaceWrapper.qml文件中的onStoreChanged事件处理函数中，storeNormalWidth的更新应该在store更新后立即进行，以避免潜在的逻辑错误。

是否建议立即修改: 是

deepin-ci-robot commented 5 months ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: justforlxz, ZhongYic00

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/treeland/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 / treeland

Winlayout restore on output changes #290

deepin pr auto review