fix: dim labels and reposition note for webhook-driven readonly fields#148
fix: dim labels and reposition note for webhook-driven readonly fields#148R-Hart80 wants to merge 1 commit into
Conversation
|
Warning Rate limit exceeded
You’ve run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✨ 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 |
207119a to
c9d1614
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
When a project is webhook-driven, the four GitHub integration input
fields (repo owner, repo name, branch, file path) were already marked
readOnly and visually styled as disabled on the inputs themselves.
This commit extends the visual treatment to their labels by applying
opacity-60 when isWebhookDriven is true, and moves the explanatory
note ("Repository fields are managed by the GitHub integration.")
to appear after all four fields so it clearly covers the whole group.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
c9d1614 to
1e51ffb
Compare
Closes #5
Problem
When
isWebhookDrivenis true, the four GitHub-integration input fields (repo_owner,repo_name,branch,git_ontology_path) were already rendered withreadOnlyandcursor-not-allowed opacity-60on the inputs themselves. However, their labels remained at full opacity, which was visually inconsistent — the fields looked disabled but the labels did not.Additionally, the explanatory note "Repository fields are managed by the GitHub integration." was placed between the first two-column grid (repo owner + name) and the second grid (branch + file path), making it ambiguous whether it applied to all four fields or only the bottom two.
Changes
app/projects/[id]/settings/page.tsxisWebhookDriven && "opacity-60"to theclassNameof all four field labels so they dim in sync with their inputs.Test plan
isWebhookDriven === true).🤖 Generated with Claude Code