Add favicon to tmt docs

[happz]

Pull Request Checklist

• implement the feature

[psss]

Nice! Thanks for the improvement. Thinking a bit about the icon size: Currently the image is 700x700 and the size is 432K, so the icon is larger then the page content :) But it will be cached, I guess, so probably not worth it?

[happz]

Nice! Thanks for the improvement. Thinking a bit about the icon size: Currently the image is 700x700 and the size is 432K, so the icon is larger then the page content :) But it will be cached, I guess, so probably not worth it?

It will be cached for sure, and we definitely can create a smaller one :)

[psss]

Ok :) Adding here: teemtee/docs#5

[happz]

@psss done.

[happz]

Documentation-only, not waiting for functional tests to complete.

[martinhoyer]

hah, I totally missed this and doing the same lol.
https://github.com/martinhoyer/tmt/tree/favicon-logo

@psss is the logo available in other formats or gimp files?

@happz you're using way to large resolution for the favicon here I'm afraid. Also not sure if all browsers are happy when it's not .ico.

[martinhoyer]

See the .ico from #3072 (on the left) vs tmt.readthedocs.io/en/latest/

[happz]

@martinhoyer Yeah, I didn't bother, I wouldn't even notice the image looks a bit weird on my screen. I guess it's like the sound quality, at some point my human ear don't hear the difference :)

[martinhoyer]

@martinhoyer Yeah, I didn't bother, I wouldn't even notice the image looks a bit weird on my screen. I guess it's like the sound quality, at some point my human ear don't hear the difference :)

/me hiding his audio equipment 👀