Skip to content

trivial: adding favicon image - #3208

Closed
taw00 wants to merge 1 commit into
dashpay:developfrom
taw00:favicon-addition
Closed

trivial: adding favicon image#3208
taw00 wants to merge 1 commit into
dashpay:developfrom
taw00:favicon-addition

Conversation

@taw00

@taw00 taw00 commented Nov 19, 2019

Copy link
Copy Markdown

noticed that someone already contributed all my pixmaps I have been using for
my builds (thank you, whoever you were). The favicon was left out. Adding
that as well. It is useful from a completist perspective and gives users a
resource they can just use if they have need of a favicon for their project.

- noticed that someone already contributed all my pixmaps I have been using for
  my builds (thank you, whoever you were). The favicon was left out. Adding
  that as well. It is useful from a completist perspective and have users a
  resource they can just use if they have need of a favicon for their project.
@nmarley

nmarley commented Nov 19, 2019

Copy link
Copy Markdown

Thanks for the PR! Is this for the Debian packaging or something? I guess I don't understand what the reason for these pixmaps is.

If this is just a generic image for websites, would it be better if it were transparent? Also wonder if it would be better on the website graphics page or here in the repo (since it isn't used in the code). Just thinking out loud, will let other devs chime in.

@taw00

taw00 commented Nov 19, 2019

Copy link
Copy Markdown
Author

It's from a completion perspective. Traditionally, favicons are not transparent, though browsers do support that. If someone wants to utilize a transparent icon, they can just use the dash32.png file.

I just added in a separate PR all of my high contrast icons (see #3209) it has an explanation for how these icons should be used by the linux desktop. I build for RPM based (Fedora and RHEL/CentOS) but it works the same for all distros.

Side note: To make the desktop configuration complete, there really needs to be a dash-qt.appdata.xml file appropriately deployed as well. I would add that to the dashpay/dash, but (a) I would have to alter mine a bit, and (b) I am not sure where would be an appropriate place for it to land. If you want to read more there are some links in this file: https://github.com/taw00/dashcore-rpm/blob/master/SOURCES/dashcore-0.14-contrib/linux/desktop/dash-qt.appdata.xml

@taw00

taw00 commented Nov 19, 2019

Copy link
Copy Markdown
Author

Also note, I will have no heartache if you ditch this addition. It's the most trivial of trivialities. ;) The other PR is more impactful. I would love to remove the need to ship those myself.

On second thought: Yeah, kill it. The website already has favicons deployed, if someone wants to do something similar, they can just use those.

@taw00

taw00 commented Nov 19, 2019

Copy link
Copy Markdown
Author

Killing this myself. It's redundant and not useful to the application.

@taw00 taw00 closed this Nov 19, 2019
@taw00
taw00 deleted the favicon-addition branch November 19, 2019 19:26
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.

2 participants