-
Notifications
You must be signed in to change notification settings - Fork 5
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
SketchUp favicon needs updating to same used for other SketchUp sites #21
Comments
This is either still busted or busted again ! |
Yup, I noticed that. Got a fix in the pipeline which pulls the favicon into the template itself instead of relying on external resources. |
Fix was done in this commit: fa19368 But we haven't pushed any update yet. Working on some other fixes as well. |
I think has been fixed. We now see the new blue SketchUp application icon in browser tabs. |
Actually, ... the icon has a permanent white background even when a dark mode is used. Please replace the icon here: ... with this icon: ... ie, they are all the same 32x32 icon. Although the last is a PNG. |
Update favicon.ico to same as other SketchUp sites. Ref Issue SketchUp#21
Update favicon.ico to same as other SketchUp sites. Ref Issue SketchUp#21
* Update favicon.ico to same as other SketchUp sites Update favicon.ico to same as other SketchUp sites. Ref Issue #21 * Delete Favicon.ico * Update favicon.ico to same as other SketchUp sites Update favicon.ico to same as other SketchUp sites. Ref Issue #21 * Edit Ruby.svg change red to Trimble blue. Changed all references of Ruby color #e62b3e to Trimble blue #005F9E. Ref Issue #22 * Fix invalid relative links to favicon in forum Fix invalid relative links to favicon in forum oneboxes. Ref: Issue #23
SketchUp favicon URL has changed causing error and standard "document" browser tab icon.
... 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/
The text was updated successfully, but these errors were encountered: