Skip to content

blog: pin sidebar profile avatar locally (stop hotlinking GitHub) - #257

Closed
Jesssullivan wants to merge 1 commit into
mainfrom
recut/pin-profile-avatar
Closed

blog: pin sidebar profile avatar locally (stop hotlinking GitHub)#257
Jesssullivan wants to merge 1 commit into
mainfrom
recut/pin-profile-avatar

Conversation

@Jesssullivan

Copy link
Copy Markdown
Owner

Re-cut of #233 (closed in the 2026-08-12 board sweep) per operator ruling; original branch jess/pin-profile-avatar@4e6e68d.

Two-line pin of the sidebar profile avatar plus the vendored image (static/images/profile-avatar.jpg, 39067 bytes).

ProfileSidebar hotlinked https://github.com/Jesssullivan.png since #20,
so any change to the GitHub account picture instantly restyles prod.
Vendor the current avatar (460x460 JPEG as served by GitHub) into
static/images/profile-avatar.jpg and point both the compact and desktop
sidebar imgs at it. Third-party request removed; future GitHub avatar
changes no longer propagate silently.
@Jesssullivan

Copy link
Copy Markdown
Owner Author

Closed as a preserved later UI asset donor. Avatar presentation is outside the current architecture tranche; the remote branch and vendored asset remain available for a future explicitly scoped visual PR.

Jesssullivan added a commit that referenced this pull request Aug 13, 2026
…recut of #257) (#260)

* fix(ui): pin sidebar profile avatar locally, stop hotlinking GitHub (recut of #257)

ProfileSidebar hotlinked https://github.com/Jesssullivan.png since #20,
so any change to the GitHub account picture instantly restyles prod.
Vendor the current avatar (460x460 JPEG as served by GitHub) into
static/images/profile-avatar.jpg and point both the compact and desktop
sidebar imgs at it. Third-party request removed; future GitHub avatar
changes no longer propagate silently.

Recut of closed PR #257 (itself a recut of #233); donor branch
recut/pin-profile-avatar, donor commit c284e6c.

(cherry picked from commit c284e6c)

* test(ui): assert pinned sidebar avatar
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.

1 participant