trigger.dev icon indicating copy to clipboard operation
trigger.dev copied to clipboard

fix(docs): tooltip color contrast accessibility

Open falbech opened this issue 2 months ago • 2 comments

Closes #2633

✅ Checklist

  • [x] I have followed every step in the contributing guide
  • [x] The PR title follows the convention.
  • [x] I ran and tested the code works

Testing

  1. Start the docs locally with pnpm run dev --filter docs
  2. In the docs, go to /manual-setup.
  3. Hover the Copy or any other action icon on any code block.

Changelog

Added a new global style.css file. Included a CSS selector that improves text visibility by adjusting the tooltip’s text color for better contrast.


Screenshots

Before:

image

After

image

falbech avatar Nov 10 '25 02:11 falbech

⚠️ No Changeset found

Latest commit: 26d677816f0bfdf40667dfd3e34191e5aa7a8bdc

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

changeset-bot[bot] avatar Nov 10 '25 02:11 changeset-bot[bot]

Walkthrough

This pull request adds a changeset documenting a patch release and includes a CSS modification. Specifically, it introduces a changelog entry for "trigger.dev" describing an accessibility fix for tooltip color contrast on documentation pages. The corresponding CSS change adds a styling rule targeting a tooltip element that follows a button, setting its text color to black to improve visual contrast.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • The changeset file is metadata documentation only
  • The CSS modification is a single, straightforward rule addition with no logic complexity
  • Changes are minimal, focused, and low-risk

Pre-merge checks and finishing touches

✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: fixing tooltip color contrast accessibility in docs, which matches the primary objective of the changeset.
Description check ✅ Passed The PR description follows the template structure with all required sections completed: issue reference, checklist items checked, testing steps provided, changelog entry, and before/after screenshots included.
Linked Issues check ✅ Passed The code changes directly address issue #2633 by adding CSS styling to improve tooltip text color contrast on the docs /manual-setup page, exactly as required.
Out of Scope Changes check ✅ Passed All changes are narrowly scoped to the stated objective: the changeset file and CSS stylesheet modification are both directly related to fixing the tooltip accessibility issue.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • [ ] Create PR with unit tests
  • [ ] Post copyable unit tests in a comment

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.

❤️ Share

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

coderabbitai[bot] avatar Nov 10 '25 02:11 coderabbitai[bot]