Skip to content

fix: utilizza il logo configurato come favicon - #1886

Open
sajotrei wants to merge 1 commit into
devcode-it:masterfrom
sajotrei:fix/configurable-favicon
Open

fix: utilizza il logo configurato come favicon#1886
sajotrei wants to merge 1 commit into
devcode-it:masterfrom
sajotrei:fix/configurable-favicon

Conversation

@sajotrei

Copy link
Copy Markdown

Descrizione

L'impostazione Logo menu quadrato / favicon viene già caricata e utilizzata per il logo compatto del menu, ma il tag <link rel="icon"> continua a puntare sempre al favicon statico predefinito.

Questa modifica:

  • mantiene assets/dist/img/favicon.png come fallback;
  • usa il file configurato in Logo menu quadrato / favicon anche come favicon del browser;
  • rimuove il MIME type image/x-icon forzato, perché il file configurabile è gestito come media e può non essere un ICO.

Compatibilità

Nessun cambiamento per le installazioni che non configurano un favicon personalizzato.

La modifica è volutamente limitata a include/top.php e non interviene sulla gestione di ROOTDIR, BASEURL, manifest o routing.

Verifiche

  • diff limitato a un solo file;
  • fallback invariato quando l'impostazione è vuota;
  • stesso file utilizzato per logo menu compatto e favicon quando configurato.

Correlato alla discussione #1642, senza modificare la logica dei percorsi applicativi.

@sajotrei
sajotrei marked this pull request as ready for review August 10, 2026 01:25
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

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