Favicon url returns a 403

meshy / pythonwheels

Adoption analysis of Python Wheels: https://pythonwheels.com/

BSD 2-Clause "Simplified" License

102 stars 26 forks source link

Favicon url returns a 403 #55

Closed ghickman closed 7 years ago

ghickman commented 9 years ago

http://pythonwheels.com/favicon.ico

There isn't a favicon in the repo so it's either been lost to the mists of time or there never was one.

Should we add one or should we remove the link?

meshy commented 9 years ago

Hey dude :) thanks for the report.

I don't think that there ever was one. It'd be nice for it to be a green circle .ico, but maybe we could get fancy, and try to create one to reflect the current state of the wheel.

I can imagine that happening in one of two ways:

we could generate it using canvas (client-side) on page load
we could generate it (somehow) on server-side as part of the update script

It's a shame we can't use the existing svg, as there seems to be very low browser support for that.

ghickman commented 9 years ago

@meshy – both good ideas, I vote for the green circle as a first step and then enhancement $later to the fancy generated one.

meshy commented 9 years ago

Sounds good to me :)

meshy commented 8 years ago

inkscape and favico.js might both be worth investigating, as browser support (as yet) remains low.

hugovk commented 8 years ago

First step in PR https://github.com/meshy/pythonwheels/pull/79.

Second step in PR #81.

ghickman commented 7 years ago

Closing with many thanks to @hugovk!