Skip to content

fix(admin): polish UX démo — copier/email stylés, stats seedbox masquées#7

Merged
ohugonnot merged 1 commit into
masterfrom
fix/demo-admin-ux
Jun 20, 2026
Merged

fix(admin): polish UX démo — copier/email stylés, stats seedbox masquées#7
ohugonnot merged 1 commit into
masterfrom
fix/demo-admin-ux

Conversation

@ohugonnot

Copy link
Copy Markdown
Owner

Résumé

  • Copier le lien : fallback execCommand('copy') pour les contextes HTTP (clipboard API nécessite HTTPS) + toast visuel — supprime le prompt() natif
  • Envoyer par email : modale stylée promptModal() (overlay flouté + card animée) au lieu du prompt() natif
  • Fix saut d'animation email : la modale utilisait sheetPop qui contient translate(-50%,-50%) — incompatible avec un card flex-centré (→ saut visible à l'apparition). Remplacé par @keyframes promptPop (scale + opacity uniquement)
  • Champ max-téléchargements : spinner natif retiré via -webkit-appearance: none + classe sheet-num-input, espacement corrigé avec le bouton
  • Onglet Statistiques : masqué hors mode seedbox — les métriques rtorrent n'ont aucun sens dans une install sans seedbox

Test

  • Demo : http://199.231.187.166:8282/share/admin.php — créer un lien, copier, envoyer par email
  • Vérifier que l'onglet Statistiques n'apparaît pas (pas de seedbox sur la démo)
  • Vérifier l'absence de saut à l'ouverture de la modale email

- copier le lien : fallback execCommand (HTTP) + toast, supprime le prompt() natif
- envoyer par email : modale stylée (promptModal) au lieu du prompt natif
- fix saut de centrage email : animation scale-only (promptPop) au lieu de sheetPop
  qui contenait translate(-50%,-50%) incompatible avec un card flex-centré
- champ max-téléchargements : spinner natif retiré, espacement corrigé (sheet-num-input)
- onglet Statistiques masqué hors mode seedbox (sans rtorrent il n'a aucun sens)
@ohugonnot ohugonnot merged commit f06f125 into master Jun 20, 2026
8 checks passed
@ohugonnot ohugonnot deleted the fix/demo-admin-ux branch June 20, 2026 17:24
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.

1 participant