Skip to content

fix(view): simplify AlbumTitle setChecked and sync on GStatus change#447

Merged
wyu71 merged 1 commit into
linuxdeepin:masterfrom
wyu71:master
Jun 30, 2026
Merged

fix(view): simplify AlbumTitle setChecked and sync on GStatus change#447
wyu71 merged 1 commit into
linuxdeepin:masterfrom
wyu71:master

Conversation

@wyu71

@wyu71 wyu71 commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Replace switch-case with direct boolean assignments in setChecked. Add Connections to react to GStatus.currentCollecttionViewIndexChanged, and component.onCompleted to initialize button state.

将setChecked从switch-case简化为直接布尔赋值。
新增Connections响应GStatus.currentCollecttionViewIndexChanged, 添加component.onCompleted初始化按钮状态。

Log: 简化AlbumTitle视图按钮状态同步逻辑
Influence: AlbumTitle的收藏视图切换按钮状态与GStatus实时同步,初始化时也能正确显示选中状态。

Replace switch-case with direct boolean assignments in setChecked.
Add Connections to react to GStatus.currentCollecttionViewIndexChanged,
and component.onCompleted to initialize button state.

将setChecked从switch-case简化为直接布尔赋值。
新增Connections响应GStatus.currentCollecttionViewIndexChanged,
添加component.onCompleted初始化按钮状态。

Log: 简化AlbumTitle视图按钮状态同步逻辑
Influence: AlbumTitle的收藏视图切换按钮状态与GStatus实时同步,初始化时也能正确显示选中状态。

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry @wyu71, you have reached your weekly rate limit of 500000 diff characters.

Please try again later or upgrade to continue using Sourcery

@deepin-ci-robot

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: pengfeixx, wyu71

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

@wyu71 wyu71 merged commit 9fe21ab into linuxdeepin:master Jun 30, 2026
16 checks passed
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