SketchUp / sketchup-yard-template

YARD Template used to generate SketchUp Ruby API documentations
MIT License
13 stars 5 forks source link

SketchUp favicon needs updating to same used for other SketchUp sites #21

Open DanRathbun opened 6 years ago

DanRathbun commented 6 years ago

SketchUp favicon URL has changed causing error and standard "document" browser tab icon.

Per Caroline in the public forum

This is the link: https://www.sketchup.com/sites/all/themes/sketch_www/favicon.ico

... it is currently ...

https://help.sketchup.com/sites/all/themes/sketch_help/favicon.ico

... which gives a 404 Not Found error.


Reference public forum thread: https://forums.sketchup.com/t/favicon-seems-to-be-missing/57625/

DanRathbun commented 4 years ago

This is either still busted or busted again !

thomthom commented 4 years ago

Yup, I noticed that. Got a fix in the pipeline which pulls the favicon into the template itself instead of relying on external resources.

thomthom commented 4 years ago

Fix was done in this commit: https://github.com/SketchUp/sketchup-yard-template/commit/fa19368c40e2525c38d67c1a4bc800e408022183

But we haven't pushed any update yet. Working on some other fixes as well.

DanRathbun commented 3 years ago

I think has been fixed. We now see the new blue SketchUp application icon in browser tabs.

DanRathbun commented 3 years ago

Actually, ... the icon has a permanent white background even when a dark mode is used.

Please replace the icon here: "https://ruby.sketchup.com/favicon.ico"

... with this icon: "https://developer.sketchup.com/sites/developer.sketchup.com/files/Favicon.ico" or this: "https://www.sketchup.com/themes/sketchup_www_terra/favicon.ico" or this: "https://aws1.discourse-cdn.com/sketchup/optimized/3X/c/7/c746725cdbfbabf601da75a1af32622133cfacdd_2_32x32.png"

... ie, they are all the same 32x32 icon. Although the last is a PNG.