Skip to content

feat(dock-plugin): add disconnect button on hover for plugin items#480

Open
MyLeeJiEun wants to merge 1 commit into
linuxdeepin:masterfrom
MyLeeJiEun:fork-from-master-0715-story-41277
Open

feat(dock-plugin): add disconnect button on hover for plugin items#480
MyLeeJiEun wants to merge 1 commit into
linuxdeepin:masterfrom
MyLeeJiEun:fork-from-master-0715-story-41277

Conversation

@MyLeeJiEun

Copy link
Copy Markdown
  1. Add CommonTextButton widget with rounded background and hover/press states
  2. Show "Disconnect" text button on hover for connected plugin items
  3. Hide connect icon on hover and restore it on leave
  4. Route disconnect click signal to the new button instead of the icon
  5. Update translations for the "Disconnect" string

Log: Connected plugin items now show a "Disconnect" button on hover.
Influence: Users can clearly see how to disconnect a dock plugin.

feat(dock-plugin): 插件项悬停时显示断开连接按钮

  1. 新增 CommonTextButton 控件,支持圆角背景和悬停/按下状态
  2. 连接的插件项在鼠标悬停时显示"断开连接"文字按钮
  3. 悬停时隐藏连接图标,离开后恢复显示
  4. 将断开点击信号连接到新按钮而非图标
  5. 更新各语言翻译,添加"断开连接"字符串

Log: 连接的插件项在悬停时显示"断开连接"按钮。
PMS: STORY-41277
Influence: 用户能清晰知道如何断开插件连接。

@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 @MyLeeJiEun, your pull request is larger than the review limit of 150000 diff characters

@deepin-ci-robot

Copy link
Copy Markdown

Hi @MyLeeJiEun. Thanks for your PR.

I'm waiting for a linuxdeepin member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@MyLeeJiEun
MyLeeJiEun force-pushed the fork-from-master-0715-story-41277 branch 2 times, most recently from 118b3c7 to c9480e8 Compare July 16, 2026 03:47
Comment thread plugins/dde-dock/common/commontextbutton.cpp Outdated
1. Add CommonTextButton widget with rounded background and hover/press states
2. Show "Disconnect" text button on hover for connected plugin items
3. Hide connect icon on hover and restore it on leave
4. Route disconnect click signal to the new button instead of the icon
5. Update translations for the "Disconnect" string

Log: Connected plugin items now show a "Disconnect" button on hover.
Influence: Users can clearly see how to disconnect a dock plugin.

feat(dock-plugin): 插件项悬停时显示断开连接按钮

1. 新增 CommonTextButton 控件,支持圆角背景和悬停/按下状态
2. 连接的插件项在鼠标悬停时显示"断开连接"文字按钮
3. 悬停时隐藏连接图标,离开后恢复显示
4. 将断开点击信号连接到新按钮而非图标
5. 更新各语言翻译,添加"断开连接"字符串

Log: 连接的插件项在悬停时显示"断开连接"按钮。
PMS: STORY-41277
Influence: 用户能清晰知道如何断开插件连接。
@MyLeeJiEun
MyLeeJiEun force-pushed the fork-from-master-0715-story-41277 branch from c9480e8 to 4bbd07d Compare July 16, 2026 12:58
@deepin-ci-robot

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: 18202781743, MyLeeJiEun

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants