Skip to content

feat: aggiunta impostazione ritardo apertura tooltip dashboard#1771

Merged
lucasalva87 merged 2 commits intomasterfrom
feature/tooltip-delay-setting
Feb 26, 2026
Merged

feat: aggiunta impostazione ritardo apertura tooltip dashboard#1771
lucasalva87 merged 2 commits intomasterfrom
feature/tooltip-delay-setting

Conversation

@lucasalva87
Copy link
Contributor

@lucasalva87 lucasalva87 commented Feb 26, 2026

Descrizione

Aggiunta una nuova impostazione 'Ritardo apertura tooltip dashboard (ms)' nella sezione Dashboard di zz_settings, che permette agli amministratori di configurare il ritardo in millisecondi prima che il tooltip venga mostrato al passaggio del mouse sugli eventi del calendario.

Motivazione

In precedenza il ritardo di apertura dei tooltip era fisso a 300ms e non configurabile. Questa modifica consente di adattare il comportamento in base alle preferenze dell'utente o alle esigenze di performance.

Modifiche apportate

update/2_11.sql

  • Inserimento della nuova impostazione Ritardo apertura tooltip dashboard (ms) nella tabella zz_settings con:
    • Valore predefinito: 300 (ms) — mantiene il comportamento preesistente
    • Tipo: integer
    • Sezione: Dashboard
  • Inserimento delle traduzioni in zz_settings_lang per italiano (id_lang=1) e inglese (id_lang=2)

modules/dashboard/edit.php

  • Aggiunta della proprietà tooltip_delay all'oggetto JavaScript globals.dashboard, valorizzata dinamicamente tramite setting('Ritardo apertura tooltip dashboard (ms)')
  • Sostituzione del valore fisso delay: 300 con il valore dinamico delay: globals.dashboard.tooltip_delay nella configurazione di Tooltipster

Compatibilità

Il valore predefinito è 300 ms, identico al valore precedentemente hardcoded, quindi il comportamento rimane invariato per le installazioni esistenti che non modificano l'impostazione.

Aggiunta una nuova impostazione 'Ritardo apertura tooltip dashboard (ms)'
nella sezione Dashboard di zz_settings, che permette di configurare
il ritardo in millisecondi prima che il tooltip venga mostrato al
passaggio del mouse sugli eventi del calendario.

- update/2_11.sql: INSERT dell'impostazione con valore predefinito 300ms
  e relative traduzioni in italiano e inglese in zz_settings_lang
- update/2_5.php: aggiunta traduzione inglese per il meccanismo di
  localizzazione delle impostazioni esistenti
- modules/dashboard/edit.php: esposizione del valore come
  globals.dashboard.tooltip_delay e utilizzo dinamico nel
  parametro 'delay' della configurazione tooltipster
@lucasalva87 lucasalva87 deleted the feature/tooltip-delay-setting branch February 26, 2026 12:55
@lucasalva87 lucasalva87 restored the feature/tooltip-delay-setting branch February 26, 2026 14:57
@lucasalva87 lucasalva87 reopened this Feb 26, 2026
@lucasalva87 lucasalva87 merged commit d4185c1 into master Feb 26, 2026
3 checks passed
@lucasalva87 lucasalva87 deleted the feature/tooltip-delay-setting branch February 26, 2026 15:06
@kilo-code-bot
Copy link

kilo-code-bot bot commented Feb 26, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)
  • modules/dashboard/edit.php - Adds configurable tooltip delay setting
  • update/2_11.sql - Database migration for new setting

Review Notes

This PR adds a configurable tooltip delay setting for the dashboard calendar, replacing the hardcoded 300ms delay with a user-configurable value.

Changes:

  • Added tooltip_delay to globals.dashboard object (line 411)
  • Updated tooltipster delay to use the configurable value (line 764)
  • Created database setting with default value of 300ms
  • Added Italian and English translations

Security Analysis:

  • ✅ No SQL injection risk - hardcoded values in SQL
  • ✅ No XSS risk - value sanitized with intval()
  • ✅ Type-safe - setting type is 'integer'

Code Quality:

  • ✅ Minimal, focused changes
  • ✅ Follows existing codebase patterns
  • ✅ Default value matches previous hardcoded value
  • ✅ Properly formatted migration with translations

The implementation is clean and ready for merge.

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