Nam Pham Dinh ThanhandClaude Opus 5
2b90492994
Bốn mục trong Cài đặt tách thành widget riêng dưới presentation/settings/:
general_settings_widget.py ngôn ngữ, giao diện, khay, gợi ý
provider_settings_widget.py provider, base URL, key, model + 2 nút nền
parameter_settings_widget.py đính kèm, cấu trúc, giới hạn sandbox
routing_settings_widget.py Auto Model Routing
Mỗi widget tự dựng control, tự nạp giá trị, tự có apply_to(data). Dialog chỉ
còn lắp ráp và gọi apply_to lúc lưu — _save từ 34 dòng xuống còn phần khung.
Làm lưới an toàn trước khi bóc: tests/ui/test_settings_dialog_dac_ta.py, 7
bài đặc tả hành vi hiện tại (mục nào có mặt, nạp đúng giá trị gì, lưu ghi vào
đúng ô nào, đổi % sang phân lẻ, xoá cache sau lưu). Bóc xong cả 7 vẫn xanh,
và trong lúc bóc chúng đã đỏ đúng hai lần ở chỗ đáng đỏ.
Đây là repo chưa từng có test Qt nào — thêm tests/ui/conftest.py dựng
QApplication offscreen. Offscreen là bắt buộc chứ không phải cho nhanh: máy
dev là máy làm việc thật, test bật cửa sổ lên là nó nhảy ra che màn hình.
Dọn kèm:
* bỏ vòng "dựng vào layout rồi lại gỡ ra" của mục Chung, cùng widget cao 0px
làm mốc cuộn — không cần nữa khi mục đó tự là một widget
* bỏ _select_combo, _secret, _model_combo, _with_load và 4 hàm provider khác
đã chuyển vào widget (127 dòng)
* bỏ 5 import chết theo (Dict, QSizePolicy, PROVIDER_LABELS, SegmentedControl,
LANGUAGES)
Giữ cầu tương thích: self.routing_*, self.prov_*, self.attach_* … thành
property trỏ vào widget con, vì 5 checker trong tools/ đọc thẳng tên cũ. Bỏ
được khi tools/ chuyển sang đọc self._provider_page.
Hai điều KHÔNG làm, ghi lại để khỏi tưởng là quên:
1. Plan ghi 4 widget và có tên `connector`. Thực tế UI connector đã dời khỏi
Cài đặt từ trước (ghi chú ở settings_dialog.py:180 bản cũ), nên số mục thật
là 5, không phải 4, và không có mục nào tên connector. Bốn mục bóc ra là 4
mục có thật; mục Bảo mật sandbox để nguyên trong dialog lần này.
2. Còn ~108 dòng chết của MS365 (_refresh_ms365_status, _ms365_sign_in,
_show_ms365_device_code, _ms365_sign_out): đọc self.ms365_status,
self.ms365_signin_btn, self.ms365_signout_btn — ba thuộc tính KHÔNG BAO GIỜ
được gán, và không hàm nào có người gọi. Gọi vào là AttributeError. Chưa
xoá vì đó là quyết định của anh Nam, không phải việc kèm theo của T07.
437 test xanh. check_dialogs, check_no_hscroll, check_design_parity đều qua.
Kèm docs/refactor/tin-gui-team-hoa.md — tin báo Hoa về platform/ -> adapters/
và bản vá Windows của AtomicJsonFile.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
74 lines
3.1 KiB
Python
74 lines
3.1 KiB
Python
"""Mục Chung trong Cài đặt — R08-T07.
|
|
|
|
Ngôn ngữ, giao diện, khay hệ thống, và dòng gợi ý cuối trang.
|
|
|
|
Bản trước khi tách dựng mục này thành một ``QFormLayout`` rời, gắn vào layout
|
|
gốc, rồi ở đoạn lắp ráp lại gỡ ra để nhét vào hộp riêng — kèm một widget cao
|
|
0 pixel làm mốc cuộn. Vòng vo đó chỉ tồn tại vì mục này không phải group box
|
|
như bốn mục kia. Gói thành widget là hết: nó tự là một trang, không cần gỡ ra
|
|
gắn vào, không cần mốc giả.
|
|
"""
|
|
from __future__ import annotations
|
|
|
|
from PySide6.QtWidgets import QFormLayout, QLabel, QVBoxLayout, QWidget
|
|
|
|
from ...i18n import LANGUAGES, tr
|
|
from ...ui.widgets import SegmentedControl, ToggleSwitch
|
|
|
|
|
|
class GeneralSettingsWidget(QWidget):
|
|
def __init__(self, ctx, parent=None):
|
|
super().__init__(parent)
|
|
data = ctx.config.data
|
|
outer = QVBoxLayout(self)
|
|
outer.setContentsMargins(0, 0, 0, 0)
|
|
form = QFormLayout()
|
|
|
|
self.language_combo = SegmentedControl()
|
|
for key, label in LANGUAGES.items():
|
|
self.language_combo.addItem(label, key)
|
|
_select(self.language_combo, ctx.config.language)
|
|
form.addRow(tr("settings.language"), self.language_combo)
|
|
|
|
# Giao diện cũng có trên hàng tài khoản ở thanh bên (một cú bấm để lật
|
|
# nhanh); đây là cùng một giá trị, nhưng có tên và có giải thích, cho
|
|
# người đi tìm nó trong Cài đặt.
|
|
self.theme_combo = SegmentedControl()
|
|
for key in ("system", "dark", "light"):
|
|
self.theme_combo.addItem(tr(f"settings.theme_{key}"), key)
|
|
_select(self.theme_combo, getattr(ctx.config, "theme", "system"))
|
|
form.addRow(tr("settings.theme"), self.theme_combo)
|
|
|
|
tray = data.get("tray", {})
|
|
self.tray_chk = ToggleSwitch(tr("settings.tray_keep"))
|
|
self.tray_chk.setChecked(bool(tray.get("minimize_on_close", True)))
|
|
form.addRow("", self.tray_chk)
|
|
self.notify_chk = ToggleSwitch(tr("settings.tray_notify"))
|
|
self.notify_chk.setChecked(bool(tray.get("notify_on_done", True)))
|
|
form.addRow("", self.notify_chk)
|
|
|
|
outer.addLayout(form)
|
|
|
|
note = QLabel(tr("settings.tip"))
|
|
note.setObjectName("hint")
|
|
note.setWordWrap(True) # không thì đúng một dòng này quyết định bề ngang dialog
|
|
outer.addWidget(note)
|
|
outer.addStretch(1)
|
|
|
|
# ---- lưu ------------------------------------------------------------
|
|
|
|
def apply_to(self, data: dict) -> None:
|
|
data["language"] = self.language_combo.currentData()
|
|
# MainWindow._open_settings áp lại giao diện sau khi dialog đóng, nên
|
|
# ghi giá trị ở đây là đủ để nó có hiệu lực.
|
|
data["theme"] = self.theme_combo.currentData()
|
|
tray = data.setdefault("tray", {})
|
|
tray["minimize_on_close"] = self.tray_chk.isChecked()
|
|
tray["notify_on_done"] = self.notify_chk.isChecked()
|
|
|
|
|
|
def _select(combo, value: str) -> None:
|
|
i = combo.findData(value)
|
|
if i >= 0:
|
|
combo.setCurrentIndex(i)
|