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 icons used in project #5

Closed
Ashark opened this issue Jun 29, 2021 · 6 comments
Closed

Fix icons used in project #5

Ashark opened this issue Jun 29, 2021 · 6 comments
Assignees
Labels
enhancement New feature or request question Further information is requested

Comments

@Ashark
Copy link
Contributor

Ashark commented Jun 29, 2021

Currently in TermplateManager.desktop the icon for main file is favorite. Probably should instead use the project's one TemplateManagerIcon.png. I guess it should be placed at some standard location (do not yet explored which exactly) by install.sh and then use its name in Icon= field.
Specifying absolute path to png works (in Dolphin the icon of the .desktop file itself changes), but using $HOME variable there does not work. So I am not sure for now how to implement it.
This icon is referenced also in install.sh and in uninstall.sh.

Some other icons are missing in my system. For example, xapp-favorite-symbolic in cuttlefish shows nothing. It is referenced in TemplateManager.desktop and in TemplateManagerTools.sh.

@gmm96
Copy link
Owner

gmm96 commented Jun 29, 2021

To be honest, I have tried using the project's icon in the main service menu specifying absolute path to png file, but it always displays the icon corresponding to the first action in the menu (template creator). This is only happening on the service menu, you can properly see the icon navigating to the directory that keeps the service menu files using Dolphin. I don't know if it is currently a bug of Dolphin.

I was looking for a tool to check if certain icon exists in system, but without success. Maybe, I should use more common icons, like the ones used in template editor (edit) and template eraser (delete). Favorite icon may solve this issue.

@gmm96
Copy link
Owner

gmm96 commented Jun 29, 2021

Well, command kf5-config --path icon is currently working in my system, so I suppose I can give it another try installing the icon in one of these directories.
On the other hand, I don't understand why some kf5-config --path arguments goes in singular (icon) and others in plural (templates), but it is ok.

@gmm96
Copy link
Owner

gmm96 commented Jun 29, 2021

After another look, xdg-icon-resource seems to be the right command to (un)install icons in system.

@Ashark
Copy link
Contributor Author

Ashark commented Jun 29, 2021

I don't know if it is currently a bug of Dolphin.

Probably yes, I also experience this. I did not yet searched or reported in bugs.kde.org

I was looking for a tool to check if certain icon exists in system

You mean at runtime of the script? Or for you at develop time to choose an icon? If latter, then use Cuttlefish application.

@gmm96
Copy link
Owner

gmm96 commented Jul 1, 2021

You mean at runtime of the script? Or for you at develop time to choose an icon? If latter, then use Cuttlefish application.

I mean at runtime, so I would have alternatives to show if one icon is not installed in a system.
Well, I'm gonna move to more common icons (favorite, edit, delete), which I expect they'll be already installed in every system, and then will try custom png icons.

@gmm96 gmm96 self-assigned this Jul 1, 2021
@gmm96
Copy link
Owner

gmm96 commented Jul 2, 2021

Added more common icons in this https://github.com/gmm96/KDE-5-Template-Manager/commit/9c38b8693c04c207177042fa09030a4305d0c185. Closing this issue.

@gmm96 gmm96 closed this as completed Jul 2, 2021
@gmm96 gmm96 pinned this issue Jul 2, 2021
@gmm96 gmm96 unpinned this issue Jul 2, 2021
@gmm96 gmm96 added enhancement New feature or request question Further information is requested labels Jul 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants