Skip to content

fix: use text color for list markers instead of accent color#17739

Open
sentry-junior[bot] wants to merge 1 commit into
masterfrom
fix/ol-marker-color
Open

fix: use text color for list markers instead of accent color#17739
sentry-junior[bot] wants to merge 1 commit into
masterfrom
fix/ol-marker-color

Conversation

@sentry-junior
Copy link
Copy Markdown
Contributor

@sentry-junior sentry-junior Bot commented May 12, 2026

Problem

Ordered list numbers in doc pages are styled with var(--accent) — the same purple used for links — making them appear clickable when they aren't.

Reported in [docs] User Feedback: Numbers appear clickable but are not.

Fix

Change marker:text-[var(--accent)]marker:text-[var(--gray-12)] in the doc page prose container so list markers use the regular body text color instead of the link/accent color.

Before / After

  • Before: ol numbers render in accent purple (looks like a link)
  • After: ol numbers render in --gray-12 (matches body text)

One-line change in src/components/docPage/index.tsx.

PR opened on behalf of @m1ch1.

The ordered list markers (numbers) were styled with var(--accent), the
same purple used for links, making them appear clickable when they are
not. Switch to var(--gray-12) to match the surrounding body text.
@vercel
Copy link
Copy Markdown

vercel Bot commented May 12, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
develop-docs Ready Ready Preview, Comment May 12, 2026 8:21pm
sentry-docs Ready Ready Preview, Comment May 12, 2026 8:21pm

Request Review

coolguyzone added a commit that referenced this pull request May 12, 2026
## Problem

Ordered list numbers in doc pages are styled with `var(--accent)` — the
same purple used for links — making them appear clickable when they
aren't.

Reported in [docs] User Feedback: Numbers appear clickable but are not.

## Fix

Two changes to the `marker:` utilities on the doc page prose container
in `src/components/docPage/index.tsx`:

1. **Color:** `marker:text-[var(--accent)]` →
`marker:text-[var(--darkPurple)]` — matches h1/heading color
2. **Weight:** adds `marker:font-normal` — sets `font-weight: 400` to
match the mockup design

- **Light mode:** markers render in `#1F1633` (dark purple, same as
headings) at weight 400
- **Dark mode:** `--darkPurple` is overridden to `var(--foreground)` via
the `.dark` rule in `globals.css`, staying consistent with headings

## Alternative

See also #17739 which uses `--gray-12` (body text color) instead.

PR opened on behalf of @m1ch1.

---------

Co-authored-by: getsentry-bot <bot@getsentry.com>
Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: Alex Krawiec <coolguyzone@users.noreply.github.com>
Co-authored-by: Alex Krawiec <alex.krawiec@sentry.io>
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.

0 participants