ui: 优化平台收藏按钮的点击反馈

Author: zensoku142

tests/test_provider_i18n.py

@@ -81,6 +81,7 @@ def test_provider_picker_text_and_numbered_count_are_localized(monkeypatch, lang
         "全部平台", "已配置", "常用平台", "未配置", "API 用量", "AI 平台列表", "删除", "已移除",
         "没有匹配的平台，试试其他名称或查看全部平台",
         "Enter 切换 · Ctrl+D 收藏 · Esc 关闭",
+        "收藏", "取消收藏", "已收藏 {provider}", "已取消收藏 {provider}",
     )
     for source in sources:
         assert source in PROVIDER_MESSAGES

tests/test_provider_picker.py

@@ -238,6 +238,57 @@ def test_star_click_toggles_favorite_without_activating_card(picker, picker_stat
     assert "elevenlabs" in state["pinned_providers"]
     assert pins_changed.count() == 1 and activated.count() == 0
     assert picker.currentData() == "codex" and picker.popup.isVisible()
+    assert picker.hint.text() == "已收藏 ElevenLabs"
+    QTest.mouseClick(picker.grid.viewport(), Qt.MouseButton.LeftButton, pos=star_center)
+    assert picker.hint.text() == "已取消收藏 ElevenLabs"
+
+
+@pytest.mark.parametrize("theme", ["light", "dark"])
+def test_star_hover_press_and_drag_out_feedback(picker, picker_state, theme, tmp_path):
+    configure_theme(APP, theme)
+    open_all(picker)
+    picker.search.setText("elevenlabs")
+    APP.processEvents()
+    viewport = picker.grid.viewport()
+    rect = picker.grid.visualItemRect(picker.grid.item(0))
+    star_center = QPoint(rect.right() - 23, rect.top() + 26)
+    QTest.mouseMove(viewport, rect.center())
+    normal = viewport.grab().toImage()
+    QTest.mouseMove(viewport, star_center)
+    hovered = viewport.grab().toImage()
+    assert viewport.cursor().shape() == Qt.CursorShape.PointingHandCursor
+    assert hovered != normal
+    QTest.mousePress(viewport, Qt.MouseButton.LeftButton, pos=star_center)
+    pressed = viewport.grab().toImage()
+    assert pressed != hovered
+    picker.popup.grab().save(str(tmp_path / f"favorite-{theme}-pressed.png"))
+    QTest.mouseMove(viewport, rect.center())
+    QTest.mouseRelease(viewport, Qt.MouseButton.LeftButton, pos=rect.center())
+    assert "elevenlabs" not in provider_picker.pinned_provider_ids()
+    assert picker.currentData() == "codex" and picker.popup.isVisible()
+    assert viewport.grab().toImage() == normal
+
+
+def test_star_release_without_press_does_not_toggle(picker):
+    open_all(picker)
+    picker.search.setText("elevenlabs")
+    rect = picker.grid.visualItemRect(picker.grid.item(0))
+    QTest.mouseRelease(picker.grid.viewport(), Qt.MouseButton.LeftButton,
+                       pos=QPoint(rect.right() - 23, rect.top() + 26))
+    assert "elevenlabs" not in provider_picker.pinned_provider_ids()
+
+
+def test_disabled_star_explains_why_it_cannot_be_pinned(picker, monkeypatch):
+    monkeypatch.setattr(provider_picker.config_manager, "get",
+                        lambda key, default=None: ["elevenlabs"] if key == "DISABLED_PROVIDER_IDS" else default)
+    open_all(picker)
+    picker.search.setText("elevenlabs")
+    rect = picker.grid.visualItemRect(picker.grid.item(0))
+    QTest.mouseClick(picker.grid.viewport(), Qt.MouseButton.LeftButton,
+                     pos=QPoint(rect.right() - 23, rect.top() + 26))
+    assert "elevenlabs" not in provider_picker.pinned_provider_ids()
+    assert "重新启用" in picker.hint.text()
+    assert picker.popup.isVisible()
 
 
 def test_shortcuts_show_three_pins_and_emit_correct_id(picker_state):

ui/provider_picker.py

@@ -21,6 +21,7 @@
     QStyle,
     QStyledItemDelegate,
     QToolButton,
+    QToolTip,
     QVBoxLayout,
     QWidget,
 )
@@ -48,6 +49,14 @@ def pinned_provider_ids() -> list[str]:
 
 
 class _ProviderCardDelegate(QStyledItemDelegate):
+    hovered_pin = None
+    pressed_pin = None
+
+    @staticmethod
+    def pin_rect(rect):
+        # 绘制和命中检测共用 40px 热区，避免只有星形细线附近才能明确操作。
+        return QRectF(rect).adjusted(rect.width() - 44, 6, -4, -rect.height() + 46)
+
     def paint(self, painter, option, index) -> None:
         provider_id, connected, pinned = index.data(Qt.ItemDataRole.UserRole)
         tokens = current_theme()
@@ -66,14 +75,22 @@ def paint(self, painter, option, index) -> None:
         icon_rect.setSize(QSize(26, 26))
         provider_icon(provider_id, 26).paint(painter, icon_rect)
         # 收藏采用矢量星形；品牌标记始终来自独立的品牌 SVG 资产。
-        center = QPointF(rect.right() - 19, rect.top() + 22)
+        star_rect = self.pin_rect(option.rect)
+        hovered_pin = self.hovered_pin == provider_id
+        pressed_pin = hovered_pin and self.pressed_pin == provider_id
+        if pinned or hovered_pin:
+            painter.setPen(QPen(QColor(tokens.accent), 1) if hovered_pin else Qt.PenStyle.NoPen)
+            painter.setBrush(QColor(tokens.accent if pressed_pin else tokens.accent_soft))
+            painter.drawRoundedRect(star_rect.adjusted(3, 3, -3, -3), 7, 7)
+        center = star_rect.center()
         points = QPolygonF([
             QPointF(center.x() + (7 if n % 2 == 0 else 3.1) * cos(-pi / 2 + n * pi / 5),
                     center.y() + (7 if n % 2 == 0 else 3.1) * sin(-pi / 2 + n * pi / 5))
             for n in range(10)
         ])
-        painter.setPen(QPen(QColor(tokens.accent if pinned else tokens.subtext), 1.1))
-        painter.setBrush(QColor(tokens.accent) if pinned else Qt.BrushStyle.NoBrush)
+        star_color = QColor(tokens.window if pressed_pin else tokens.accent if pinned or hovered_pin else tokens.subtext)
+        painter.setPen(QPen(star_color, 1.4))
+        painter.setBrush(star_color if pinned else Qt.BrushStyle.NoBrush)
         painter.drawPolygon(points)
         painter.setFont(option.font)
         painter.setPen(QColor(tokens.value))
@@ -196,6 +213,9 @@ def _set_filter(self, selected: str) -> None:
         self._populate()
 
     def _populate(self, *_args) -> None:
+        self.grid.itemDelegate().hovered_pin = None
+        self.grid.itemDelegate().pressed_pin = None
+        self._grid_viewport.unsetCursor()
         query = self.search.text().strip().casefold()
         if query and self._filter != "all":
             # 搜索面向所有可接入平台，不能因默认的已配置筛选而隐藏用户正在找的品牌。
@@ -352,7 +372,10 @@ def _toggle_pin(self, item: QListWidgetItem) -> None:
         if self._pins != pins:
             bind_text(self.hint, "常用平台保存失败，请检查数据目录")
             return
-        bind_text(self.hint, "Enter 切换 · Ctrl+D 收藏 · Esc 关闭")
+        bind_text(self.hint, lambda: tr(
+            "已收藏 {provider}" if provider_id in pins else "已取消收藏 {provider}",
+            provider=provider_short_name(provider_id),
+        ))
         self.pins_changed.emit()
 
     def eventFilter(self, watched, event) -> bool:
@@ -370,17 +393,40 @@ def eventFilter(self, watched, event) -> bool:
         if watched is self._grid_viewport and event.type() == QEvent.Type.Resize:
             self.grid.doItemsLayout()
             self._position_remove_buttons()
+        if watched is self._grid_viewport and event.type() == QEvent.Type.Leave:
+            self.grid.itemDelegate().hovered_pin = None
+            self._grid_viewport.unsetCursor()
+            self._grid_viewport.update()
         if watched is self._grid_viewport and event.type() in (
+            QEvent.Type.MouseMove, QEvent.Type.ToolTip,
             QEvent.Type.MouseButtonPress, QEvent.Type.MouseButtonRelease,
-        ) and event.button() == Qt.MouseButton.LeftButton:
-            item = self.grid.itemAt(event.position().toPoint())
-            if item is not None:
-                rect = self.grid.visualItemRect(item)
-                star = rect.adjusted(rect.width() - 44, 5, -4, -rect.height() + 44)
-                if star.contains(event.position().toPoint()):
-                    if event.type() == QEvent.Type.MouseButtonRelease:
+        ):
+            position = event.pos() if event.type() == QEvent.Type.ToolTip else event.position().toPoint()
+            item = self.grid.itemAt(position)
+            delegate = self.grid.itemDelegate()
+            over_pin = item is not None and delegate.pin_rect(self.grid.visualItemRect(item)).contains(position)
+            provider_id = item.data(Qt.ItemDataRole.UserRole)[0] if over_pin else None
+            delegate.hovered_pin = provider_id
+            self._grid_viewport.setCursor(Qt.CursorShape.PointingHandCursor if over_pin else Qt.CursorShape.ArrowCursor)
+            self._grid_viewport.update()
+            if event.type() == QEvent.Type.ToolTip and over_pin:
+                label = "请先在全部平台中重新启用此平台。" if provider_id in self._disabled else "取消收藏" if provider_id in self._pins else "收藏"
+                QToolTip.showText(event.globalPos(), tr(label), self._grid_viewport)
+                return True
+            if event.type() == QEvent.Type.MouseButtonPress and event.button() == Qt.MouseButton.LeftButton:
+                delegate.pressed_pin = provider_id
+                if over_pin:
+                    return True
+            if event.type() == QEvent.Type.MouseButtonRelease and event.button() == Qt.MouseButton.LeftButton:
+                pressed_pin = delegate.pressed_pin
+                delegate.pressed_pin = None
+                # 必须在同一收藏热区按下并松开；拖出按钮时取消，不能误收藏或切换平台。
+                if pressed_pin is not None:
+                    if pressed_pin == provider_id:
                         self._toggle_pin(item)
                     return True
+                if over_pin:
+                    return True
         if event.type() == QEvent.Type.KeyPress:
             if (watched is self.grid or watched in self.remove_buttons.values()) and event.key() == Qt.Key.Key_Delete:
                 self._remove_selected()

ui/provider_translations.py

@@ -121,6 +121,16 @@
     "已连接": ("Connected", "已連線", "接続済み", "연결됨"),
     "已配置": ("Configured", "已設定", "設定済み", "설정됨"),
     "常用平台": ("Favorites", "常用平台", "お気に入り", "즐겨찾기"),
+    "收藏": ("Add to favorites", "收藏", "お気に入りに追加", "즐겨찾기에 추가"),
+    "取消收藏": ("Remove from favorites", "取消收藏", "お気に入りから削除", "즐겨찾기에서 제거"),
+    "已收藏 {provider}": (
+        "Added {provider} to favorites", "已收藏 {provider}",
+        "{provider} をお気に入りに追加しました", "{provider} 즐겨찾기에 추가됨",
+    ),
+    "已取消收藏 {provider}": (
+        "Removed {provider} from favorites", "已取消收藏 {provider}",
+        "{provider} をお気に入りから削除しました", "{provider} 즐겨찾기에서 제거됨",
+    ),
     "未配置": ("Not configured", "未設定", "未設定", "설정되지 않음"),
     "API 用量": ("API usage", "API 用量", "API 使用量", "API 사용량"),
     "AI 平台列表": ("AI provider list", "AI 平台清單", "AI プロバイダー一覧", "AI 공급업체 목록"),