Полный переход на CTk-диалоги с поддержкой темной/светлой темы & Исправление белой темы в подсказках (тулпитах) в трей-настройках by Yan4ik000 · Pull Request #1011 · Flowseal/tg-ws-proxy

Yan4ik000 · 2026-06-19T05:02:16Z

Добавлен новый модуль - ui/ctk_dialogs.py.

В нем все функции вызова любых по типу CTk окон, обычные сообщения - show_info, ошибки - show_error, вопросы "Да/Нет" - ask_yes_no и окно загрузки обновления с прогрессом - run_with_progress (оно теперь одно из компонентов CTk окон).

В итоге, на Windows (неважно на 11, 10 или 7 винде) это добавляет полную поддержку темной темы в программу, теперь включая темную тему, она распространится не только на основное окно трей-настроек, но и на все диалоговые окна/ошибки/предупреждения и тому подобные окна.
При этом поддержка светлой темы никуда не исчезла.

Пока что, на всякий случай, если по какой либо причине будет ошибка при отрисовке CTk сообщений, оставлен fallback на стандартные win32 уведомления/диалоги.

Также пофикшен баг, или недочет (не знаю как назвать), когда при включенной светлой теме, подсказки под параметрами в настройках оставались в темной теме.

Если хотите сами протестировать работу проги, можете скачать артефакт из actions в моем форке

Скрины:

(на начало в заголовках не обращайте внимания, я запрашивал уведомления напрямую через скрипт, а не получал их нормальным способом, и из-за этого писалось в начале заголовка - "dlg-X", в обычном использовании выглядит примерно так)

Исправленное окно подсказок при белой теме:

Flowseal · 2026-06-19T08:43:18Z

+    if kind == "error":
+        line = max(6, int(canvas * 0.13))
+        inset = int(canvas * 0.31)
+        draw.line((inset, inset, canvas - inset, canvas - inset), fill=(255, 255, 255, 255), width=line)
+        draw.line((canvas - inset, inset, inset, canvas - inset), fill=(255, 255, 255, 255), width=line)
+    else:
+        bbox = draw.textbbox((0, 0), text, font=font)
+        x = (canvas - (bbox[2] - bbox[0])) / 2 - bbox[0]
+        y = (canvas - (bbox[3] - bbox[1])) / 2 - bbox[1] + canvas * 0.01
+        draw.text((x, y), text, font=font, fill=(255, 255, 255, 255))


Плохое решение делать kind строкой

Flowseal · 2026-06-19T08:43:37Z

+def _appearance_is_dark(ctk: Any) -> bool:
+    try:
+        return str(ctk.get_appearance_mode()).lower() == "dark"
+    except Exception:
+        return False


по-моему где-то уже есть эта функция

…rk_mode

Yan4ik000 · 2026-06-19T09:42:14Z

'kind' заменён на 'str, Enum'
Относительное дублирование было, в ui/ctk_tooltip.py на 59 строчке, "if ctk.get_appearance_mode() == "Dark":"
Я просто вынес эту логику в общую функцию "is_dark_mode" в файле ui/ctk_theme.py

Flowseal · 2026-06-23T12:18:17Z

+            _run_ctk_modal(lambda: ctk_run_with_progress(
+                ctk,
+                parent=None,
+                theme=ctk_theme_for_platform(),
+                title="TG WS Proxy — Обновление",
+                message="Устанавливается обновление...",
+                icon_path=ICON_PATH,
+                task=_do_update,
+            ))


Дедлок, если будет ошибка. Новому окну с ошибкой нужен будет ввод, но он будет заблокирован из-за grab_set формы с прогрессом. Форма с прогрессом будет ждать завершения perfom_update, а оно будет ждать закрытия окна ошибки. Ошибки для run_with_progress лучше показывать через set_status, как это было реализовано до.

Я бы оставил единое окно с кнопками прямо в run_with_progress, так как никаким больше образом это окно использоваться не будет, сохранив прежнюю логику полностью

Flowseal · 2026-06-23T12:20:36Z


+def _run_ctk_modal(callback):
+    if ctk is not None and ensure_ctk_thread(ctk, _config.get("appearance", "auto")):
+        if threading.current_thread().name == "ctk-root":


Магическое значение ctk-root

Flowseal · 2026-06-23T12:21:40Z

+        from utils.tray_common import ensure_ctk_thread, ctk_run_dialog
+
+        if ensure_ctk_thread(ctk, "auto"):
+            if threading.current_thread().name == "ctk-root":


Магическое значение ctk-root

Лучше в tray_common сделать проверку с вынесением ctk-root в константу из этих проверок и ensure_ctk_thread

Flowseal · 2026-06-23T12:28:43Z

 from ui.ctk_tooltip import attach_ctk_tooltip, attach_tooltip_to_widgets

 log = logging.getLogger('tg-mtproto-proxy')
+_APP_ICON_PATH = str(Path(__file__).resolve().parent.parent / "icon.ico")


В windows.py уже есть ICON_PATH
Стоит унифицировать

Flowseal · 2026-06-23T12:31:31Z

+    root, body, content_frame, scaling = _build_dialog_base(ctk, parent, theme, title, message, kind, icon_path)
+
+    dark = is_dark_mode(ctk)
+    footer_bg = "#2b2b2b" if dark else "#f3f3f3"


Магические значения. Используйте CtkTheme

Flowseal · 2026-06-23T12:32:13Z

+
+    if icon_path:
+        try:
+            root.after(200, lambda: root.iconbitmap(icon_path))


Сделайте 300мс

Flowseal · 2026-06-23T12:36:56Z

+    body_bg = "#2b2b2b" if dark else "#ffffff"
+    text_color = theme.text_primary
+
+    root = ctk.CTkToplevel(parent) if parent is not None else ctk.CTkToplevel()


эквивалентно CTkToplevel(parent)

Yan4ik000 · 2026-06-23T13:40:05Z

Исправлено

Для обновления вернул логику одного окна. Ошибки теперь показываются через status label внутри run_with_progress, без открытия отдельного окна. Обновление статуса из worker-потока сделано через очередь, чтобы не трогать Tk напрямую из другого потока
Вынес ctk-root в CTK_THREAD_NAME и добавил is_ctk_thread() в utils.tray_common
Унифицировал путь к иконке через APP_ICON_PATH в utils.tray_common
Убрал магические цвета в CTk-диалогах/tooltip, теперь используются значения из CtkTheme
Заменил задержку установки iconbitmap на 300 мс
Упростил создание окна до CTkToplevel(parent).

Yan4ik000 · 2026-06-23T14:15:40Z

Плюсом я конфликт в windows.py исправил после мержа pr #1025, и также у меня не запускалось приложение после сборки с решенным конфликтом, из-за того что .json локализацию забыли в конфиг и pyinstaller положить, починил и это

Flowseal · 2026-06-23T16:54:38Z

-    datas=[(ctk_path, 'customtkinter/')] + gi_datas + typelib_datas,
+    datas=[
+        (ctk_path, 'customtkinter/'),
+        (os.path.join(os.path.dirname(SPEC), os.pardir, 'ui', 'i18n', '*.json'), 'ui/i18n'),
+    ] + gi_datas + typelib_datas,


Не надо, исправлено в 6b5fd72

Flowseal · 2026-06-23T16:54:42Z

-    datas=[],
+    datas=[
+        (os.path.join(os.path.dirname(SPEC), os.pardir, 'ui', 'i18n', '*.json'), 'ui/i18n'),
+    ],


Не надо, исправлено в 6b5fd72

Flowseal · 2026-06-23T16:54:46Z

-    datas=[(ctk_path, 'customtkinter/')],
+    datas=[
+        (ctk_path, 'customtkinter/'),
+        (os.path.join(os.path.dirname(SPEC), os.pardir, 'ui', 'i18n', '*.json'), 'ui/i18n'),
+    ],


Не надо, исправлено в 6b5fd72

Yan4ik000 · 2026-06-23T18:35:28Z

откатил

Yan4ik000 added 2 commits June 19, 2026 07:05

Переход на кастомные CTk-диалоги с поддержкой темной/светлой темы

a439773

Добавлена поддержка белой темы подсказкам в настройках (тулпитам)

d15564e

Yan4ik000 requested a review from Flowseal as a code owner June 19, 2026 05:02

Flowseal requested changes Jun 19, 2026

View reviewed changes

fix (Flowseal#1011): замена kind на 'str, Enum' и вынос утилиты is_da…

ebca2d5

…rk_mode

Yan4ik000 requested a review from Flowseal June 19, 2026 09:42

Flowseal requested changes Jun 23, 2026

View reviewed changes

fix (Flowseal#1011): доработан update-flow и унифицированы CTk-хелперы

e609e3e

Yan4ik000 requested a review from Flowseal June 23, 2026 13:42

Исправлен конфликт в windows.py в Flowseal#1011

277fead

Flowseal requested changes Jun 23, 2026

View reviewed changes

Merge remote-tracking branch 'upstream/main'

34b4c71

Yan4ik000 force-pushed the main branch from 66e41a7 to 34b4c71 Compare June 23, 2026 18:34

Yan4ik000 requested a review from Flowseal June 23, 2026 18:35

Uh oh!

Conversation

Yan4ik000 commented Jun 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Добавлен новый модуль - ui/ctk_dialogs.py.

Скрины:

Исправленное окно подсказок при белой теме:

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Yan4ik000 commented Jun 19, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Yan4ik000 commented Jun 23, 2026

Uh oh!

Yan4ik000 commented Jun 23, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Yan4ik000 commented Jun 23, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Yan4ik000 commented Jun 19, 2026 •

edited

Loading