Skip to content

YNU-756 & YNU-757: Fix broken favicon and edit page links#109

Merged
ihsraham merged 2 commits intomasterfrom
fix/favicon
Feb 13, 2026
Merged

YNU-756 & YNU-757: Fix broken favicon and edit page links#109
ihsraham merged 2 commits intomasterfrom
fix/favicon

Conversation

@ihsraham
Copy link
Collaborator

What

Back in Sept 2025 (commit 40b54de), a bulk asset update accidentally replaced the real favicon.ico with an SVG placeholder. Browsers couldn't parse it, so the tab icon was broken on all pages. The "Edit this page" link also still pointed to the default Docusaurus template repo.

Changes

  • Replaced the placeholder favicon.ico with the actual Yellow Network favicon (pulled from yellow.org)
  • Fixed the edit link URL to point to github.com/layer-3/docs instead of the Docusaurus template repo
  • Renamed "Edit this page" to "Suggest a fix" (via i18n translation override)

Testing

Verified locally that:

  • Favicon loads correctly on all pages, including direct links with anchors
  • "Suggest a fix" link opens the correct .mdx file in the layer-3/docs repo on GitHub

The favicon.ico was an SVG placeholder masquerading as an .ico file,
causing browsers to reject it. This resulted in a broken/missing favicon,
especially noticeable when navigating directly to anchor URLs.

Replaced with the actual Yellow Network favicon from yellow.org (128x128 PNG).
… a fix'

- Changed editUrl from the default Docusaurus template repo to
  https://github.com/layer-3/docs/tree/master/
- Renamed 'Edit this page' label to 'Suggest a fix' via i18n/en/code.json
- Added i18n translation files generated by write-translations
@ihsraham ihsraham requested a review from dpatsora as a code owner February 13, 2026 10:11
@alessio alessio self-requested a review February 13, 2026 12:09
@ihsraham ihsraham added this pull request to the merge queue Feb 13, 2026
Merged via the queue into master with commit bc75542 Feb 13, 2026
7 checks passed
@ihsraham ihsraham deleted the fix/favicon branch February 13, 2026 12:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants