Lift the admin footer text off the grey it could not be read against - #3377
Conversation
grey-400 measures 2.58:1 on the white admin body. The footer text is 12px, so it is normal text and answers to 4.5:1; 500 is the lightest stop on the ramp that clears it, at 4.97:1. The social icons take the same step. The social links also had no hover feedback: their resting rule and .frm_wrap a:hover are both (0,2,1) and this file loads later, so the grey won. The added pseudo-class settles it at 600, and :focus-visible rides along so the affordance is not mouse-only. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
Warning Review limit reachedNext included review available in 4 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (2)
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 |
 |
|
Overall Grade |
Security Reliability Complexity Hygiene |
Code Review Summary
| Analyzer | Status | Updated (UTC) | Details |
|---|---|---|---|
| PHP |  |
Sep 18, 2026 2:30p.m. | Review ↗ |
| JavaScript | |
Sep 18, 2026 2:30p.m. | Review ↗ |
Important
AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.
2 participants
--grey-400measures 2.58:1 on the white admin body. The footer text is 12px, so it is normal text and answers to 4.5:1 —--grey-500is the lightest stop on the ramp that clears it, at 4.97:1. The social icons take the same step.The social links also had no hover feedback: their resting rule and
.frm_wrap a:hoverare both (0,2,1) and this file loads later, so the grey won. The added pseudo-class settles it at 600, with:focus-visiblealongside so the affordance is not mouse-only.Token bumps only — no palette values changed. Built CSS diff is three footer rules and nothing else.
PROD-5662's missing-accessible-name half was already fixed on master in
8bb0bee28; this covers the contrast half.🤖 Generated with Claude Code