FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fix: utilizza il logo configurato come favicon by sajotrei · Pull Request #1886 · devcode-it/openstamanager · GitHub

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

Copy link
Copy Markdown
Contributor

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 marked this pull request as ready for review August 10, 2026 01:25

Copy link
Copy Markdown

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

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL