Skip to content

Conversation

@BLumia
Copy link
Member

@BLumia BLumia commented Nov 11, 2024

调整 AppGroupModel 与 AMAppItemModel 的关系,其它部分暂未调整。

变化说明:

  1. 目前决定复用 AppGroupManager,后续可能需要补充一些实现(目前没处理分组内容为空后清除分组的行为,也没自动为所有未分组的应用分配分组)
  2. AppGroupManager 不再是单例
  3. AMAppItem 和 AppGroup 现在都是 AppItem 的子类了,便于后续直接组合对应的两个模型
  4. AMAppsModel 不再提供分组信息(后续在组合模型提供,分组信息的数据源仍由 AppGroupManager 管理)

后续待调整内容:

  1. 暴漏的 appsModel 转为一个组合模型,同时包含这两个模型的数据
  2. 处理应用组的初始化、位置信息更新等行为,确保每个应用总是有分组信息

@BLumia BLumia requested a review from tsic404 November 11, 2024 13:12
调整 AppGroupModel 与 AMAppItemModel 的关系,其它部分暂未调整。

后续待调整内容:

1. 暴漏的 appsModel 转为一个组合模型,同时包含这两个模型的数据
2. 处理应用组的初始化、位置信息更新等行为,确保每个应用总是有分组信息
@BLumia BLumia force-pushed the app-group-refactor branch from 706dd85 to af9ca5a Compare November 12, 2024 09:10
@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: BLumia, tsic404

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

@BLumia BLumia merged commit 2cd53dc into linuxdeepin:master Nov 12, 2024
@BLumia BLumia deleted the app-group-refactor branch November 12, 2024 09:22
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.

3 participants