Use system icon theme and increase icon size

[ilya-fedin]

Can help with #3216, but for larger icons App::wnd()->iconWithCounter should be reworked :(
Fixes #1908

[23rd]

Suggested change

	? (muted ? "telegram-mute-panel" : "telegram-attention-panel")
	? qsl("telegram-%1-panel").arg(muted ? "mute" : "attention")

[ilya-fedin]

fixed

[john-preston]

Thanks!

[AndydeCleyre]

Will icons in the ticons folder, if found, still take precedence?

[ilya-fedin]

No, they will be used only on appindicator

[AndydeCleyre]

@ilya-fedin isn't the appindicator what I'm talking about? In my case I mean the systray icon on plasma desktop.

[ilya-fedin]

No, appindicator is used only on gnome, pantheon and unity currently

[varlesh]

@ilya-fedin Скажу по-русски, боюсь знания моего енглиша не хватит.
Во-первых, это похвально за интеграцию трея и системных значок, но юзать сейчас это не удобно. Во-вторых неужели QSystemTray не умеет в HiDPI? Зачем лочить размер значка в 48px, ведь для большинства юзеров (те, кто не использует HiDPI-экраны) значок будет размыт, потому что 48px уменьшается до 22px (это дефолтный размер практически во всех ДЕ). И как ранее писали проблема HiDPI скорее всего кроется в png, думаю с svg все проблемы уйдут. Так же у плазмы есть опция скаллинга PLASMA_USE_QT_SCALING и для QT5 тоже QT_AUTO_SCREEN_SCALE_FACTOR, возможно стоит о них подумать как о хаке или обходе.

Третье, значок badge захардкорен. Выглядит это странно - половина работает, половина нет. Смысл тогда в такой интеграции? Может добавить кнопку отключения этого badge и вместо нее юзать обычный telegram-attention-panel? (извиняюсь, нашел опцию)

[ilya-fedin]

Скажу по-русски, боюсь знания моего енглиша не хватит.

Дай телеграм, не будем народ смущать. Мой - @IlyaFedin.
Возможно, на некоторые вопросы ответит этот пост.

[github-actions]

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.