-
-
Notifications
You must be signed in to change notification settings - Fork 79
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: move icons to gressource #242
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for working on this!
The change looks good to me - makes sense to bundle them as resources.
It seems like you've changed the formatting for exm-rating.c
. I'm not sure if there is any change here, or if it's just been formatted by accident, but please stick to the project's coding style.
Oh sorry, it's because I thought that I need to change something but i was wrong |
Fixed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few other small issues I noticed
Once fixed, could you squash this into a single commit please to keep git history tidy?
Let's get this merged for 0.4, thanks for the contribution! |
Thanks |
Icons are currently installed in datadir, it can cause conflict with others apps (maybe they are using the same icons). So, i move icons to
gressource