Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix kde5 trayicon #155

Open
wants to merge 2 commits into
base: experimental
Choose a base branch
from

Conversation

PavelSann
Copy link
Contributor

Исправил отображение иконки в трее для KDE 5 Plasma

В KDE 5 Plasma не отображается иконка QSystemTrayIcon если
в QIcon задан путь к файлу иконки.
Чтобы иконка показывалась нужно использовать имя иконки из темы:
QIcon::fromTheme("mytetra")
Иконка должна быть в папке /usr/share/icons/ или ~/.local/share/icons/

PavelSann and others added 2 commits January 24, 2021 15:02
В KDE 5 Plasma не отображается иконка QSystemTrayIcon если
в QIcon задан путь к файлу иконки.
Чтобы иконка показывалась нужно использовать имя иконки из темы:
QIcon::fromTheme("mytetra")
Иконка должна быть в папке /usr/share/icons/ или ~/.local/share/icons/
@Shura0
Copy link

Shura0 commented Dec 20, 2021

Спасибо, помогло

@xintrea
Copy link
Owner

xintrea commented Dec 21, 2021

Не понял, а сама тема/иконка как появится в /usr/share/icons/ или ~/.local/share/icons/ ?

@PavelSann
Copy link
Contributor Author

Копирование иконки есть в make install который генерирует qt

@PavelSann
Copy link
Contributor Author

mytetra/app/app.pro
mytetra/app/Makefile

@xintrea
Copy link
Owner

xintrea commented Apr 25, 2022

Копирование иконки есть в make install который генерирует qt

Этого недостаточно. MyTetra распространяется не только в исходниках или в linux-пакетах, а как предсобранное переносимое приложение. Все данные должны содержаться в самом бинарнике MyTetra. MyTetra не должна полагаться на то, что где-то в каком-то системном каталоге обязательно окажется определенный файл.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants