fix: match list markers to heading color instead of accent#17740
Merged
Conversation
Change marker color from var(--accent) to var(--darkPurple) so ordered list numbers match h1/h2/h3 heading color instead of the link accent. In light mode this is #1F1633 (dark purple); in dark mode it resolves to var(--foreground) via the .dark override, staying consistent with headings in both themes. Alternative to #17739 (which uses --gray-12 body text color). PR opened on behalf of @m1ch1.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
added 2 commits
May 12, 2026 21:05
Adds marker:font-normal so ordered list numbers render at weight 400, matching the mockup design alongside the --darkPurple heading color.
Switches marker:font-normal (400) to marker:font-medium (500).
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 7f27d43. Configure here.
Align document list marker styling with the mockup by using normal marker font weight instead of medium. Co-Authored-By: Claude <noreply@anthropic.com> Co-authored-by: Alex Krawiec <coolguyzone@users.noreply.github.com>
![sentry[bot]](https://avatars.githubusercontent.com/in/12637?s=60&v=4){kind=small}
| className={[ | ||
| 'pt-6 px-6 prose dark:prose-invert max-w-full text-[var(--gray-12)] prose-a:no-underline hover:prose-a:underline', | ||
| 'prose-code:font-normal prose-code:font-mono marker:text-[var(--accent)] prose-li:my-1', | ||
| 'prose-code:font-normal prose-code:font-mono marker:text-[var(--darkPurple)] marker:font-medium prose-li:my-1', |
Contributor
There was a problem hiding this comment.
Bug: The list marker font weight is set to marker:font-medium instead of the intended marker:font-normal, contradicting the PR's goal.
Severity: LOW
Suggested Fix
Change the class marker:font-medium to marker:font-normal at src/components/docPage/index.tsx:93 to align the list marker font weight with the intended design as specified in the PR description.
Prompt for AI Agent
Review the code at the location below. A potential bug has been identified by an AI
agent. Verify if this is a real issue. If it is, propose a fix; if not, explain why it's
not valid.
Location: src/components/docPage/index.tsx#L93
Potential issue: The code at `src/components/docPage/index.tsx:93` applies the
`marker:font-medium` Tailwind CSS class, resulting in a font weight of 500 for list
markers. However, the pull request description and a specific commit message
(`685eb6d1`) explicitly state the intention was to use `marker:font-normal` (font weight
400) to match a design mockup. The final committed code appears to have accidentally
retained the incorrect `font-medium` class, leading to a visual inconsistency with the
intended design.
Did we get this right? 👍 / 👎 to inform future reviews.
coolguyzone
approved these changes
May 12, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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 insrc/components/docPage/index.tsx:marker:text-[var(--accent)]→marker:text-[var(--darkPurple)]— matches h1/heading colormarker:font-normal— setsfont-weight: 400to match the mockup design#1F1633(dark purple, same as headings) at weight 400--darkPurpleis overridden tovar(--foreground)via the.darkrule inglobals.css, staying consistent with headingsAlternative
See also #17739 which uses
--gray-12(body text color) instead.PR opened on behalf of @m1ch1.