From b71fa69dc5de9cc3a5ae371e28d8e4bf1d968e4c Mon Sep 17 00:00:00 2001 From: Vidip Singh <112854574+vidipsingh@users.noreply.github.com> Date: Mon, 20 Jan 2025 13:03:17 +0530 Subject: [PATCH] Fix favicon path Fix favicon gets "not found" error on non-homepage pages. Set the path based on the root URL. Fixes #507 --- _layouts/base.html | 2 +- js/main.js | 20 +++++++++++++++++++- 2 files changed, 20 insertions(+), 2 deletions(-) diff --git a/_layouts/base.html b/_layouts/base.html index 656bc3fb..f1f9d1c2 100755 --- a/_layouts/base.html +++ b/_layouts/base.html @@ -4,7 +4,7 @@ - + {{ page.title }} diff --git a/js/main.js b/js/main.js index cc9809bc..4050c5fe 100755 --- a/js/main.js +++ b/js/main.js @@ -16,7 +16,25 @@ $(document).ready(function () { if (logoID < 10) { logoID = "0" + logoID; } - document.querySelector('#defaultIcon1').href = 'https://www.sugarlabs.org/assets/favicon_' + logoID + '.png'; + + var defaultIcon = document.querySelector('#defaultIcon1'); + if (defaultIcon) { + var logoID = colorIndex + 1; + if (logoID < 10) { + logoID = "0" + logoID; + } + + var rootUrl = ''; + var baseTag = document.querySelector('base'); + if (baseTag && baseTag.href) { + rootUrl = baseTag.href; + } else { + rootUrl = window.location.protocol + '//' + window.location.host + '/'; + } + + defaultIcon.href = rootUrl + 'assets/favicon_' + logoID + '.png'; + } + var h = document.querySelector('.logo1').innerHTML; h = h.replace(/033cd2/g, selectedColors[0]); h = h.replace(/78e600/g, selectedColors[1]);