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

Bug on displaying Icon in Application #3

Closed
armandhaa opened this issue Jan 25, 2019 · 5 comments
Closed

Bug on displaying Icon in Application #3

armandhaa opened this issue Jan 25, 2019 · 5 comments

Comments

@armandhaa
Copy link

Issue:
The app cannot find 'icon.png' ehrn running from different directory. No problem when running from Densify drectory
message:
Traceback (most recent call last):
File "./Densify/densify", line 428, in
win.set_icon_from_file("icon.png") # Set App Icon
GLib.Error: g-file-error-quark: Failed to open file 'icon.png': No such file or directory (4)

quick solution:
I have to exclude lines in app referring to showing icon.png and header.png icons to make it work / runnable from any directory

@hkdb
Copy link
Owner

hkdb commented Jun 30, 2019

Deleted my previous comment. I know what you mean now. Looks like what I should do is do full path for both icon.png and header.png. Will keep that in mind for the future release.

When I run the app with .desktop though, this is not a problem so I never even noticed before.

Thanks!

@armandhaa
Copy link
Author

armandhaa commented Jun 30, 2019 via email

@hkdb
Copy link
Owner

hkdb commented May 16, 2020

I have gone ahead and put in a fix for this in the latest release - v0.3.0. Thanks again!

@hkdb hkdb closed this as completed May 16, 2020
@hkdb hkdb reopened this May 16, 2020
@hkdb
Copy link
Owner

hkdb commented May 16, 2020

I just re-read your issue once again and realized that you might have been moving the script else where but not the png's? Even my latest fix assumes that the png's are always in the same directory as the script. So if you moved the images over to the same directory at the script, it should work. Either that or comment out the lines like you did.

Perhaps adding some error handling would be good though. Maybe just dismiss the images altogether if they don't exist automatically?

@hkdb
Copy link
Owner

hkdb commented Aug 18, 2020

Closing due to lack of response.

@hkdb hkdb closed this as completed Aug 18, 2020
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

No branches or pull requests

2 participants