Projet

Général

Profil

Amélioration #3938

Favicon

Ajouté par Marc Claverie il y a plus de 6 ans. Mis à jour il y a plus de 6 ans.

Statut:
Intégré
Priorité:
Normal
Assigné à:
Nicolas Thing-Leoh
Version cible:
Début:
16/04/2013
Echéance:
% réalisé:

100%

Version source:
Solution proposée:

Le paramètre [DYNACASE_FAVICO] existe dans CORE.
Il est utilisé dans les ZONE FDL:HTMLHEAD et CORE:HTMLHEAD.

Pour les documents, c'est l'icone de la famille qui est utilisé comme "icon" sauf pour IE ou c'est le CORE_FAVICON car il ne sait pas mettre des .png en "icon" (pas testé sur IE10 par contre).

Sur supervisor/index.html c'est "Images/dynacase.ico" qui est utilisé. C'est normal car dans ce cas d'exploitation on n'est pas sur le contexte.

Apparemment on ne peut pas ajouter un "favicon" facilement sur toutes les pages qui n'en n'ont pas.
Il faut le décrire explicitement sur la page.

Attention IE cherche aussi un favicon.ico à la RACINE du site dans le cas où il n'est pas présent sur la page. Ce problème particulier n'est pas lié à Dynacase mais aux sites web en général. Ceci peut être réglé par une modification de la configuration apache pour ne plus avoir de 404 dans les log.

Le fait de rajouter le link "icon" sur les pages de manière automatique sur les pages généré par layout peut être envisagé mais risqué. Il ne faut pas mettre de code html non voulu.

Donc la solution consiste de manière brute à rajouter
<link rel="icon" href="[DYNACASE_FAVICO]" />
<link rel="shortcut icon" href="[DYNACASE_FAVICO]" />
dans les pages de haut-niveau.
Les pages impactées sont :
loginform.xml (authent)
appswitcher
admin (interface générale)
webdesk ?

Principaux fichiers impactés:

loginform.xml

Wiki Détail:
Contrôle:
Jalons: 3.2 R0

Description

il semble que le paramètre core_favico ne soit pas utilisé partout, ce qui pose problème pour *** qui veut la personnaliser:
Voici un extrait, mais je ne suis pas sur de couvrir tous les modules.
Il faudrait faire une passe pour la 3.2.6 et les modules qui seront mis à jour en même temps.


ack dynacase.ico
dynacase-control/index.html
5:        <LINK rel="icon" href="images/icons/dynacase.ico" >
6:        <link rel="shortcut icon" href="images/icons/dynacase.ico" >
13:        <link rel="icon" type="image/png" href="images/icons/dynacase.ico" />

dynacase-onefam/Actions/onefam_manage_search.html
8:    <link rel="icon" href="CORE/Images/dynacase.ico">
9:    <link rel="SHORTCUT ICON" href="CORE/Images/dynacase.ico">

dynacase-appswitcher/Actions/APP_SWITCHER/app_switcher.html
4:    <link rel="icon" href="CORE/Images/dynacase.ico">
5:    <link rel="SHORTCUT ICON" href="CORE/Images/dynacase.ico">

dynacase-admin/Actions/CORE_ADMIN/core_admin_root.html
5:    <link rel="icon" type="image/x-icon" href="CORE/Images/dynacase.ico" />

dynacase-core/Supervisor/index.html
5:    <link rel="icon" href="Images/dynacase.ico" >

dynacase-core/Images/Core/Makefile
14:$(pubdir)/$(appname)/dynacase.ico : dynacase.ico
20:publish: $(pubdir)/$(appname)/logo32.ico $(pubdir)/$(appname)/dynacase.ico

dynacase-admin-uis/dynacase-admin-vault-ui/Actions/VAULT/vault_view.xml
3:    <link rel="icon" type="image/x-icon" href="CORE/Images/dynacase.ico">


Demandes liées

Lié à Core - Amélioration #2749: [Favicon] Mettre une favicon par défaut Rejeté 19/06/2012
Lié à App Switcher - Amélioration #3941: Favicon Intégré 16/04/2013

Historique

#1 Mis à jour par Éric Brison il y a plus de 6 ans

  • Statut changé de À analyser à Analysé
  • Assigné à Éric Brison supprimé
  • Temps estimé mis à 1.00
  • Solution proposée mis à jour (diff)

#2 Mis à jour par Éric Brison il y a plus de 6 ans

  • Principaux fichiers impactés mis à jour (diff)

#3 Mis à jour par Nicolas Thing-Leoh il y a plus de 6 ans

  • Statut changé de Analysé à Assigné
  • Assigné à mis à Nicolas Thing-Leoh

#4 Mis à jour par Nicolas Thing-Leoh il y a plus de 6 ans

  • % réalisé changé de 0 à 100

#5 Mis à jour par Éric Brison il y a plus de 6 ans

  • Statut changé de Assigné à Intégré

Appliqué par commit commit:659ee1ba48b1305d373960b080ba5723f7453718.

Formats disponibles : Atom PDF