-
Notifications
You must be signed in to change notification settings - Fork 148
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
An Icon? #54
Comments
I'd happily accept a patch, but besides the open issue of making/choosing an icons, I'm not familiar with the process of integrating and deploying icons, and I'm afraid this is not high enough in my priority list to go figure it out. I googled for CC icons and these looks nice:
In short, if a charitable soul makes a pull request to add an icon, I'll happily merge it. |
Not my area of expertise, but I don't think the xdot source is the place to add a Debian app icon. Instead, I think (?) you want to add it to the package: ...specifically, to the .desktop file, e.g. like in the discussion here: |
hi! The source is absolutely the right place. An icon belongs to the app, not to the distribution. (I'm currently updating the FreeBSD port, by the way.) Here's a good example of installing icons, desktop files and other stuff with distutils: https://0xacab.org/mat/mat/blob/4b9a65758da4bb27724ac1f94162810a29cb3877/setup.py#L44-50 data_files=[
('share/applications', ['mat.desktop']),
('share/mat', ['data/FORMATS', 'data/mat.glade']),
('share/pixmaps', ['data/mat.png']),
('share/doc/mat', ['README.md', 'README.security']),
('share/man/man1', ['mat.1', 'mat-gui.1']),
('share/nautilus-python/extensions', ['nautilus/nautilus-mat.py'])
], (I would prefer |
setuptools doesn't even have to do anything special with the icon. Just blessing a particular icon and shipping it in the tarball, would allow distros to do the right thing with it :) |
Forwarded from: Debian #854131:
I know you aren't shipping the desktop file, we are. But it would be nice if there was a project logo to use as the icon.
The text was updated successfully, but these errors were encountered: