Skip to content

fix: point the announcement banner at the Ink app - #727

Open
feafel wants to merge 1 commit into
inkonchain:mainfrom
feafel:fix/banner-links-to-app
Open

feafel wants to merge 1 commit into
inkonchain:mainfrom
feafel:fix/banner-links-to-app

Conversation

@feafel

@feafel feafel commented Nov 4, 2026

Copy link
Copy Markdown

What

One attribute in theme.config.tsx:

       <a
         className="!text-white hover:!text-white/80"
-        href="/"
+        href={URLS.appUrl}
         target="_blank"
         rel="noopener noreferrer"
       >

Why

The banner across the top of every page says "Mainnet is LIVE!" — and links to / with target="_blank", so clicking it opens a second tab of the docs site itself. Confirmed on production:

<a class="!text-white hover:!text-white/80" href="/" target="_blank" rel="noopener noreferrer">
  <span class="ink-typer ink-banner-typer" aria-label="Mainnet is LIVE!">

There is no mainnet docs page it could sensibly point at; the obvious destination is the Ink app, which is exactly what URLS.appUrl (https://inkonchain.com) holds — the same URL the footer ("App") and the navbar already link to. URLS is already imported in this file for docsRepositoryBase.

Verification

  • pnpm run build — exit 0
  • inspected the server-rendered HTML: the banner now carries href="https://inkonchain.com"
  • prettier --check theme.config.tsx and pnpm run lint:js — clean

The top banner says 'Mainnet is LIVE!' but links to '/' with
target="_blank" - clicking it opens a second tab of the docs
themselves. Verified on production:

  <a class="!text-white hover:!text-white/80" href="/" target="_blank" ...>
    <span class="ink-typer ink-banner-typer" aria-label="Mainnet is LIVE!">

Use URLS.appUrl (https://inkonchain.com), which the footer and navbar
already use for the same destination and which is already imported in
this file for docsRepositoryBase.

Verified with a full build: the server-rendered banner now carries
href="https://inkonchain.com".
@feafel
feafel requested a review from a team as a code owner November 4, 2026 21:40
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.

2 participants