Skip to content

updating favicon.ico - #163

Merged
heckj merged 5 commits into
swiftlang:mainfrom
heckj:favicon-update
Sep 8, 2026
Merged

updating favicon.ico#163
heckj merged 5 commits into
swiftlang:mainfrom
heckj:favicon-update

Conversation

@heckj

@heckj heckj commented Sep 8, 2026

Copy link
Copy Markdown
Member

Summary

use the icon from swift.org, setting it up in common/ and copying it into each catalog prior to assembling the docc archives and merging them.

  • updates both favicon.ico and favicon.svg (DocC default only supports favicon.ico)
  • favicon.svg is intentionally the inverse, intended to be used with <link rel="mask-icon">

Validation

  • build doc suite locally (./scripts/build_docs.py) success
  • Previewed locally with python3 -m http.server 8123 --directory .build-output
  • verify .build_output/main/favicon.ico is correct (Swift bird logo)
  • verify .build_output/main/favicon.svg is correct (Swift bird by itself (mask icon))

resolves #161


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@heckj heckj self-assigned this Sep 8, 2026
@heckj
heckj marked this pull request as ready for review September 8, 2026 20:45
@heckj
heckj requested a review from 0xTim September 8, 2026 20:45

@0xTim 0xTim left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Amazing!

@heckj
heckj merged commit cc71a1c into swiftlang:main Sep 8, 2026
13 checks passed
@heckj
heckj deleted the favicon-update branch September 8, 2026 21:18
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.

favicon not yet included in docs.swift.org

2 participants