docs: update banner to ScreenshotOne remove CarbonAds#1528
Merged
dinwwwh merged 9 commits intomiddleapi:mainfrom Apr 9, 2026
Merged
Conversation
dosubot
bot
added
the
size:S
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
📝 WalkthroughWalkthroughRemoved the Carbon Ads configuration from VitePress theme config and updated the site banner: changed headline text, helper span punctuation, CTA URL/text to ScreenshotOne, and banner background color from rgba(255, 0, 189, 0.8) to rgb(79, 70, 229). Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
apps/content/.vitepress/theme/components/Banner.vue (1)
94-102: Drop obsolete.banner-helperstyles.The helper span is no longer rendered, so this CSS block is now dead code.
🧹 Suggested cleanup
-.banner-helper { - display: none; -} - -@media (min-width: 768px) { - .banner-helper { - display: inline; - } -}🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@apps/content/.vitepress/theme/components/Banner.vue` around lines 94 - 102, Remove the dead CSS rules for the .banner-helper selector: delete the .banner-helper block and its media query variant (the rules setting display: none and display: inline at min-width: 768px) since the helper span is no longer rendered; search for ".banner-helper" in the project to ensure no other usages remain and remove the redundant CSS from the stylesheet containing those rules.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@apps/content/.vitepress/theme/components/Banner.vue`:
- Line 38: In the Banner.vue component update the headline string that currently
reads "The screenshot API for developers -" to remove the trailing separator;
locate the headline in the Banner component/template and delete the trailing "
-" so the copy becomes "The screenshot API for developers" (adjust surrounding
spacing/punctuation if needed to preserve layout and avoid orphaned CTA
wrapping).
---
Nitpick comments:
In `@apps/content/.vitepress/theme/components/Banner.vue`:
- Around line 94-102: Remove the dead CSS rules for the .banner-helper selector:
delete the .banner-helper block and its media query variant (the rules setting
display: none and display: inline at min-width: 768px) since the helper span is
no longer rendered; search for ".banner-helper" in the project to ensure no
other usages remain and remove the redundant CSS from the stylesheet containing
those rules.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 63215aa2-7b4d-46a4-b0e0-a9b4f603c3f2
📒 Files selected for processing (2)
apps/content/.vitepress/config.tsapps/content/.vitepress/theme/components/Banner.vue
💤 Files with no reviewable changes (1)
- apps/content/.vitepress/config.ts
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Code Review
This pull request removes Carbon Ads from the VitePress configuration and updates the site banner to promote ScreenshotOne, including changes to the text, link, and background color. Feedback was provided regarding the removal of the banner-helper span, which results in dead CSS and potential responsiveness issues on smaller screens; it is recommended to wrap the new text in that span to maintain the existing layout logic.
More templates
@orpc/ai-sdk
@orpc/arktype
@orpc/client
@orpc/contract
@orpc/experimental-durable-iterator
@orpc/hey-api
@orpc/interop
@orpc/json-schema
@orpc/nest
@orpc/openapi
@orpc/openapi-client
@orpc/otel
@orpc/experimental-pino
@orpc/experimental-publisher
@orpc/experimental-publisher-durable-object
@orpc/experimental-ratelimit
@orpc/react
@orpc/react-query
@orpc/experimental-react-swr
@orpc/server
@orpc/shared
@orpc/solid-query
@orpc/standard-server
@orpc/standard-server-aws-lambda
@orpc/standard-server-fastify
@orpc/standard-server-fetch
@orpc/standard-server-node
@orpc/standard-server-peer
@orpc/svelte-query
@orpc/tanstack-query
@orpc/trpc
@orpc/valibot
@orpc/vue-colada
@orpc/vue-query
@orpc/zod
commit: |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
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
1 participant
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.
Summary by CodeRabbit
Release Notes
Updates
Chores