Unstyled .link class sweep (follow-up to #566) #568
Labels
No labels
actions
bug
cli
duplicate
enhancement
fork
forum
git storage
help wanted
insights
invalid
issues
moderation
oidc
ownership transfer
packages
pr/merge protection rules
projects
pull requests
question
releases
sponsorships
tags
webhooks
wiki
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
crueber/walhub#568
Loading…
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Follow-up to #566, deliberately NOT bundled with it.
The .link class has zero CSS rules in shipped web/src/ui.css — Tailwind v4 preflight resets those buttons/links to plain muted text, so every .link control is visually indistinguishable from its caption. #566 fixed only the two inline-draft composer Cancel buttons (small secondary .btn treatment, option (b)) and left the other ~10 uses untouched:
Decide ONE systemic treatment (a shared .link component rule in ui.css @layer components per the #405 precedent + style-guideline entry, or per-site canonical-class composition) and apply it repo-wide. Acceptance: every remaining .link control distinguishable from surrounding text in both themes, with hover + cursor affordance.
Fixed by #571 (merged): one shared .link component rule in ui.css (accent both themes, hover underline, cursor-pointer) covering all 25 sites across 10 files, zero JSX changes; #566/#567 .btn surfaces untouched by design. Guideline §2 entry + law-12 amendment in the same commit. Verified: 1387 unit tests green (smoke excluded, pre-existing), vite/esbuild green, independent review APPROVE with census re-verified.