psa-systems/mokosh-apps
Watch
9
0
Fork
You've already forked mokosh-apps
 0
Code Issues Pull requests 1 Projects Releases 14 Packages Wiki Activity Actions

fix(ui): swap Tailwind-v4-removed ring-opacity-5 for ring-black/5 #465

Merged
nrupard merged 1 commit from fix/MAPPS-400-ring-opacity-tailwind-v4 into main 2026-08-04 19:16:09 +02:00
Conversation 0 Commits 1 Files changed 2 +4 -4
nrupard commented 2026-08-04 18:20:10 +02:00
Owner
Copy link

What

Swap the four ring-1 ring-black ring-opacity-5 sites to the Tailwind-v4 alpha-modifier form ring-1 ring-black/5.

Why

Tailwind v4 removed the ring-opacity-* utilities. With ring-opacity-5 a no-op, ring-black painted a solid black ring instead of the intended 5% hairline on: every DataTable wrapper (table.rs), the user menu, the notification panel, and the portal user menu (layout.rs). The repo already documented the identical bg-opacity-* removal at src/components/modal.rs:101.

Found by the 2026-08-04 UI audit (MAPPS-399), finding F3; confirmed against current main.

Verify

  • grep -rn ring-opacity- src returns nothing.
  • Class-string-only change; fmt/clippy/wasm build/tests unaffected (CI confirms).

Closes MAPPS-400.

## What Swap the four `ring-1 ring-black ring-opacity-5` sites to the Tailwind-v4 alpha-modifier form `ring-1 ring-black/5`. ## Why Tailwind v4 removed the `ring-opacity-*` utilities. With `ring-opacity-5` a no-op, `ring-black` painted a **solid black ring** instead of the intended 5% hairline on: every `DataTable` wrapper (`table.rs`), the user menu, the notification panel, and the portal user menu (`layout.rs`). The repo already documented the identical `bg-opacity-*` removal at `src/components/modal.rs:101`. Found by the 2026-08-04 UI audit (MAPPS-399), finding F3; confirmed against current `main`. ## Verify - `grep -rn ring-opacity- src` returns nothing. - Class-string-only change; fmt/clippy/wasm build/tests unaffected (CI confirms). Closes MAPPS-400.
fix(ui): swap Tailwind-v4-removed ring-opacity-5 for ring-black/5
All checks were successful
Check / fmt + clippy + tests (pull_request) Successful in 1m44s
Details
Create release / Create release from merged PR (pull_request) Has been skipped
Details
c7d45187cd 
Tailwind v4 removed the ring-opacity-* utilities, so `ring-1 ring-black ring-opacity-5` rendered a solid black ring instead of the intended 5% hairline on every DataTable wrapper (table.rs) and three dropdown panels: the user menu, the notification panel, and the portal user menu (layout.rs). Swap all four to the v4 alpha-modifier form `ring-black/5`. The repo already documented the equivalent bg-opacity-* removal at src/components/modal.rs. Found by the 2026-08-04 UI audit (MAPPS-399), finding F3.

#MAPPS-400

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
nrupard deleted branch fix/MAPPS-400-ring-opacity-tailwind-v4 2026-08-04 19:16:10 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
psa-systems/mokosh-apps!465
No description provided.