Skip to content

Addition of lockup icon, adjustments to logo and mark#1165

Merged
francinelucca merged 8 commits intoprimer:mainfrom
kylewaynebenson:main
Mar 18, 2026
Merged

Addition of lockup icon, adjustments to logo and mark#1165
francinelucca merged 8 commits intoprimer:mainfrom
kylewaynebenson:main

Conversation

@kylewaynebenson
Copy link
Contributor

@kylewaynebenson kylewaynebenson commented Feb 28, 2026

Mark 24

Before After
image image

Mark 16

Before was slightly stretched to pixel fit 16x16, after restores to un-stretched circle

Before After
image image

Logo 24

Icon is slightly wider to keep height match with old logotype.

Before After
image image

Logo 16

Before After
image image

Copilot AI review requested due to automatic review settings February 28, 2026 00:05
@kylewaynebenson kylewaynebenson requested a review from a team as a code owner February 28, 2026 00:05
@changeset-bot
Copy link

changeset-bot bot commented Feb 28, 2026

🦋 Changeset detected

Latest commit: e1e577c

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@primer/octicons Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

@simurai simurai left a comment

Choose a reason for hiding this comment

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

@kylewaynebenson I added a Changeset 297accf, but CI still fails. I think the following is missing:

  1. Add a keyword to keywords.json 👉 "lockup-github": ["brand", "github", "logo"],
  2. Add the React icon name (LockupGithubIcon) to public-api.test.js.snap. There might be a script to use, but maybe also fine to just add it manually.

@francinelucca francinelucca requested a review from simurai March 17, 2026 21:11
francinelucca and others added 2 commits March 17, 2026 17:11
Updated version for @primer/octicons to minor and modified the change log entries.
Copy link
Contributor

@simurai simurai left a comment

Choose a reason for hiding this comment

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

Ok, I changed the version to minor since we add an additional icon.

Otherwise, I think this is good to ship? 🚢

/cc @lukasoppermann @francinelucca

@francinelucca francinelucca enabled auto-merge (squash) March 18, 2026 02:54
@francinelucca francinelucca merged commit 63bc8d0 into primer:main Mar 18, 2026
12 checks passed
@primer primer bot mentioned this pull request Mar 18, 2026
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