Skip to content

fix: add readme codeblock bg, remove duplicated styles - #973

Merged
43081j merged 1 commit into
npmx-dev:mainfrom
mikouaji:fix/update-readme-styles
Feb 4, 2026
Merged

fix: add readme codeblock bg, remove duplicated styles#973
43081j merged 1 commit into
npmx-dev:mainfrom
mikouaji:fix/update-readme-styles

Conversation

@mikouaji

@mikouajimikouaji commented Feb 4, 2026

Copy link
Copy Markdown
Contributor

Implements #931
Also removed some duplicated code regarding the code block and its copy button

Before:
image

image

After:
image

image

@vercel

vercelBot commented Feb 4, 2026

Copy link
Copy Markdown
Contributor

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

ProjectDeploymentActionsUpdated (UTC)
npmx.devReadyReadyPreview, CommentFeb 4, 2026 10:43pm
2 Skipped Deployments
ProjectDeploymentActionsUpdated (UTC)
docs.npmx.devIgnoredIgnoredPreviewFeb 4, 2026 10:43pm
npmx-lunariaIgnoredIgnoredFeb 4, 2026 10:43pm

Request Review

@codecov

codecovBot commented Feb 4, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@coderabbitai

coderabbitaiBot commented Feb 4, 2026

Copy link
Copy Markdown
Contributor
📝 Walkthrough

Walkthrough

The pull request modifies the Readme.vue component to remove copy-button functionality and styling from code blocks. The changes eliminate all CSS rules related to the copy button UI, including hover and focus interactions. Explicit display, width, and position declarations in the .readme-code-block rule are replaced with a Tailwind @apply bg-bg-subtle; directive. The code-block base styling remains intact whilst the dedicated .readme-copy-button element styling and related transition rules are removed. No exported public entities were affected. The net change is 1 line added and 42 lines removed.

Suggested reviewers

  • danielroe
🚥 Pre-merge checks | ✅ 1
✅ Passed checks (1 passed)
Check nameStatusExplanation
Description check✅ PassedThe pull request description is directly related to the changeset, referencing issue #931 and explaining the removal of duplicate code related to code blocks and copy buttons.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Comment @coderabbitai help to get the list of available commands and usage tips.

@43081j
43081j added this pull request to the merge queueFeb 4, 2026
Merged via the queue into npmx-dev:main with commit 84d849eFeb 4, 2026
17 checks passed
Sign up for freeto 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

@mikouaji@43081j