Skip to content

Menú del compte a la capçalera, logotip i descàrrega de dades - #103

Merged
jordimas merged 6 commits into
mainfrom
feat/account-menu
Nov 1, 2026
Merged

jordimas merged 6 commits into
mainfrom
feat/account-menu

Conversation

@lequims

@lequims lequims commented Nov 1, 2026

Copy link
Copy Markdown
Member
  • Afegeix el logotip d'Arena Cat a la capçalera, enllaçat amb la pàgina d'avaluació
  • Agrupa les accions del compte (recordatoris, descàrrega de dades, baixa del compte i tancament de la sessió) en un menú desplegable amb icones, i separa la baixa de la resta d'opcions
  • Afegeix l'opció «Descarrega les meves dades», que baixa l'exportació com a fitxer JSON
  • Inclou les tasques omeses a l'exportació de dades (GET /api/auth/export)
  • Actualitza la documentació de la capçalera i de l'exportació

🤖 Generated with Claude Code

lequims and others added 5 commits November 1, 2026 16:22
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…count deletion

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
role="menu"
className="absolute right-0 z-20 mt-1 w-56 rounded-md border border-slate-200 bg-white py-1 shadow-lg"
>
<p className="border-b border-slate-200 px-4 py-2 text-sm break-all text-slate-500">

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

El menú queda tallat en pantalles estretes. Ho he reproduït amb amplades de 320, 375 i 390 px: quan la capçalera ocupa dues files, els controls queden a l’esquerra i el desplegable, amb right-0 w-56, comença a x = −80,6 px. Això deixa fora de la pantalla les icones i part dels textos. Podríem alinear els controls a la dreta quan salten de fila o ajustar la posició del desplegable perquè quedi dins de la pantalla?

type="button"
onClick={() => setOpen((value) => !value)}
aria-haspopup="menu"
aria-expanded={open}

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

El comportament del teclat no correspon als rols ARIA del menú. Ho he comprovat: Retorn obre el desplegable però manté el focus al botó, la fletxa avall no fa res i Tab permet sortir-ne sense tancar-lo. Les accions són accessibles amb Tab, però role="menu" i role="menuitem" impliquen una navegació específica amb el teclat. Podríem implementar aquesta gestió del focus i de les tecles, o simplificar-lo com un desplegable amb enllaços i botons sense aquests rols?

@jordimas

jordimas commented Nov 1, 2026

Copy link
Copy Markdown
Collaborator

Com a única millora, afegiria la capçalera Cache-Control: no-store a GET /api/auth/export, perquè els navegadors i els intermediaris no desin en memòria cau la resposta amb dades personals. És un canvi mínim; no el considero bloquejant per integrar la PR.

…and stop caching the data export

Drop the ARIA menu roles (no arrow-key navigation is implemented), close the
dropdown when focus leaves it, keep it inside the viewport and right-align the
header controls when they wrap. Send Cache-Control: no-store on /auth/export.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@lequims

lequims commented Nov 1, 2026

Copy link
Copy Markdown
Member Author

Gràcies pels comentaris! Ja els he resolt (5820b1e):

  • Menú tallat en pantalles estretes: els controls de la capçalera s'alineen a la dreta quan salten de fila i el desplegable té una amplada màxima de 100vw - 2rem, de manera que queda sempre dins de la pantalla.
  • Teclat i rols ARIA: he optat per simplificar-ho. He tret role="menu", menuitem i separator (i aria-haspopup), de manera que és un desplegable amb enllaços i botons. A més, es tanca quan el focus en surt (per exemple amb Tab) i amb Escape.
  • Cache-Control: no-store a GET /api/auth/export, amb prova i una línia a la documentació.

🤖 Generated with Claude Code

@lequims
lequims requested a review from jordimas November 1, 2026 17:57
@jordimas

jordimas commented Nov 1, 2026

Copy link
Copy Markdown
Collaborator

Gràcies!

@jordimas
jordimas merged commit 69f0d92 into main Nov 1, 2026
2 checks passed
@jordimas
jordimas deleted the feat/account-menu branch November 1, 2026 20:11
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