Skip to content

fix(ui): keep each disclosure label tied to its own summary - #132

Closed
wavival wants to merge 4 commits into
devfrom
fix/decisions-disclosure
Closed

wavival wants to merge 4 commits into
devfrom
fix/decisions-disclosure

Conversation

@wavival

@wavival wavival commented Oct 6, 2026

Copy link
Copy Markdown
Owner

Summary

  • On the project pages, the architecture decisions (nested "Ver detalle" disclosures) showed "Cerrar" while closed. Cause: the .when-open / .when-closed rules used a descendant selector, so an open ecosystem part (the first one is open) switched the labels of every closed decision inside it.
  • The rules now target the summary of the same disclosure (.disclosure > summary .when-open). Closed decisions read "Ver detalle", open ones "Cerrar", as before the ecosystem change. No markup, copy or script change.
  • Regression test in tests/ecosystem.spec.ts (it fails without the CSS fix: the closed decision's "Ver detalle" was hidden). DESIGN.md (dated 2026-10-06) and CHANGELOG.md updated.

Verification

  • format:check, lint, check, build, csp:check, css:check: pass.
  • Playwright (desktop and mobile Chromium): all tests pass, including the new one.
  • No conflicts with dev.

🤖 Generated with Claude Code

https://claude.ai/code/session_01MYK1VWRrVBpo1Hvb3wQJnB


Generated by Claude Code

claude added 2 commits October 6, 2026 13:22
The when-open and when-closed rules used a descendant selector, so an open
ecosystem part switched the closed architecture decisions nested inside it to
"Cerrar". The rules now target the summary of the same disclosure. Test and
changelog added.

Co-Authored-By: Claude Sonnet 5.5 <[email protected]>
Claude-Session: https://claude.ai/code/session_01MYK1VWRrVBpo1Hvb3wQJnB
@vercel

vercel Bot commented Oct 6, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
wavival-dev Ready Ready Preview Oct 6, 2026 1:41pm UTC

wavival commented Oct 6, 2026

Copy link
Copy Markdown
Owner Author

Closed: its changes are included in #134, the single pull request for this work.


Generated by Claude Code

This branch was successfully deployed

1 active deployment
Preview — f00f8be8 Deployed Oct 6, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants