Skip to content

Conversation

@cassidoo
Copy link
Contributor

@cassidoo cassidoo commented Feb 3, 2026

Currently the connector modal looks like this:

image

This PR fixes the label fonts on the callouts to be uniform and legible, as well as the link to the contributing guide:

image

@vercel
Copy link

vercel bot commented Feb 3, 2026

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

Project Deployment Actions Updated (UTC)
npmx.dev Ready Ready Preview, Comment Feb 3, 2026 5:08pm
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs.npmx.dev Ignored Ignored Preview Feb 3, 2026 5:08pm
npmx-lunaria Ignored Ignored Feb 3, 2026 5:08pm

Request Review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 3, 2026

📝 Walkthrough

Walkthrough

This change modifies the styling in a Vue component file. The contributor notice block has been simplified by removing extra spacing and background styling from the inner container, whilst the badge retains lean span styling. The contributor link colour has been updated from amber to blue. Additionally, the warning block header styling has been refined from a monospaced bold style to a more compact inline-block format with uppercase letter spacing and rounded corners. Structural adjustments to the contributor notice container have been made without altering functional behaviour. No changes to connection or form submission logic.

Suggested reviewers

  • danielroe
🚥 Pre-merge checks | ✅ 1
✅ Passed checks (1 passed)
Check name Status Explanation
Description check ✅ Passed The pull request description clearly relates to the changeset, describing styling improvements to the connector modal's label fonts and contributor link appearance.

✏️ 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

Important

Action Needed: IP Allowlist Update

If your organization protects your Git platform with IP whitelisting, please add the new CodeRabbit IP address to your allowlist:

  • 136.113.208.247/32 (new)
  • 34.170.211.100/32
  • 35.222.179.152/32

Failure to add the new IP will result in interrupted reviews.


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

@codecov
Copy link

codecov bot commented Feb 3, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Copy link
Member

@danielroe danielroe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚀

@danielroe danielroe enabled auto-merge February 3, 2026 17:10
@danielroe danielroe added this pull request to the merge queue Feb 3, 2026
Merged via the queue into npmx-dev:main with commit 9b356e7 Feb 3, 2026
14 checks passed
@cassidoo cassidoo deleted the cw/modal-legibility branch February 3, 2026 19:00
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