Skip to content

Icons on the buttons, saying what each one does - #9

Merged
tannevaled merged 1 commit into
mainfrom
landing-icons
Oct 8, 2026
Merged

tannevaled merged 1 commit into
mainfrom
landing-icons

Conversation

@tannevaled

Copy link
Copy Markdown
Contributor

Every button now carries an icon that says what it does:

  • GitHub's mark for GitHub links;
  • a book for documentation and references;
  • a rocket for getting started;
  • a graph for benchmarks;
  • a package for modules;
  • a globe for a live demo;
  • an external-link arrow for anything else.

Source: primer/octicons @primer/octicons-react@19.40.0, copied verbatim, never redrawn. The MIT licence is in layouts/partials/icons/OCTICONS-LICENSE.

Rendering:

  • currentColor, so each icon follows its button in light and dark;
  • aria-hidden, since the button's text already says where it leads;
  • one CSS rule beside .btn aligns them.

Checked: the site builds with hugo --gc --minify --panicOnWarning before and after the change, and the built home page carries 2 icons.

🤖 Generated with Claude Code

Every button now carries an Octicons glyph chosen by what it does: GitHub's
mark for GitHub, a book for documentation and references, a rocket for
getting started, a graph for benchmarks, a package for modules, a globe for
a live demo, and an external-link arrow otherwise. The glyphs come from
primer/octicons @primer/octicons-react@19.40.0, copied verbatim and not
redrawn. Their MIT licence travels in layouts/partials/icons/OCTICONS-LICENSE.
They are drawn in currentColor, so they follow the button in light and dark,
and they are aria-hidden, since the text already says where each button
leads. One CSS rule beside .btn aligns them.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@tannevaled
tannevaled merged commit eb226ac into main Oct 8, 2026
3 checks passed
@tannevaled
tannevaled deleted the landing-icons branch October 8, 2026 15:02
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