Development #69965
toulouse-2022: masquer le contenu du titre du header également sur iPhone
0%
Description
Le titre générale, définie via l'option de l'intégraphique (global_title) ou site_title (Portail) est masqué par le thème, au profit du titre défini par la variable toulouse_site_title.
Sur iPhone ou via epiphany-browser (qui utilisent le moteur de rendu webkit) le text-indent ne fonctionne pas comme attendu.
Fichiers
Demandes liées
Historique
Mis à jour par Nicolas Roche il y a plus d'un an
- Fichier Capture d’écran de 2022-10-06 17-29-16.png Capture d’écran de 2022-10-06 17-29-16.png ajouté
Mis à jour par Nicolas Roche il y a plus d'un an
- Fichier 0001-toulouse-2022-hide-header-title-content-69965.patch 0001-toulouse-2022-hide-header-title-content-69965.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Nicolas Roche il y a plus d'un an
- Lié à Bug #69398: Le nom du site, lorqu'il est masqué, apparaît quand même sur iPhone ajouté
Mis à jour par Corentin Séchet il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Ça n'est pas un souci pour toutes les intégrations graphiques du coup ? Parce que de mémoire, le text-indent est aussi utilisé pour masquer le texte quand un logo est défini.
Mis à jour par Benjamin Dauvergne il y a plus d'un an
On pourrait avoir une explication du changement de comportement concernant text-indent qu'on soit tous éclairés à ce sujet.
Mis à jour par Nicolas Roche il y a plus d'un an
- Fichier Capture d’écran de 2022-10-09 19-16-49.png Capture d’écran de 2022-10-09 19-16-49.png ajouté
Corentin : je ne trouve que 6 intégrations concernées (listées dans #69398)
Benjamin: je n'ai pas réussi à trouver l'explication.
En attendant #69398 et comme le bug ne semble pas encore gênant (dans les autres intégrations graphiques, le second titre ne s'intègre pas trop mal)
je propose juste ici un contournement du bug, temporairement.
Mis à jour par Corentin Séchet il y a plus d'un an
Corentin Séchet a écrit :
Ça n'est pas un souci pour toutes les intégrations graphiques du coup ? Parce que de mémoire, le text-indent est aussi utilisé pour masquer le texte quand un logo est défini.
Désolé Nico, j'avais mal regardé, j'avais pas vu le ticket lié. Pour la peine je l'ai pris, j'investiguerai ça en détail et en attendant ça me va de patcher ça comme ça pour Toulouse.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus d'un an
Attention à l'accessibilité et au SEO.
"site_title" doit apporter une information importante : le texte du logo "Toulouse Mairie - Metropole"
Le supprimer va laisser lisible uniquement l'information "Espace famille" sans localisation.
Ce qui n'est pas anodin.
Mis à jour par Nicolas Roche il y a plus d'un an
- Fichier 0001-toulouse-2022-toulouse-2022-hide-header-title-conten.patch 0001-toulouse-2022-toulouse-2022-hide-header-title-conten.patch ajouté
- Fichier Capture d’écran de 2022-10-12 10-58-01.png Capture d’écran de 2022-10-12 10-58-01.png ajouté
- Statut changé de Solution validée à Solution proposée
Merci pour l'alarme Thomas.
Et donc voici plutôt une contre proposition de Fred (tirée de https://dev.entrouvert.org/issues/69398#note-14)
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus d'un an
J'ai poussé un patch pour le core.
Si pas le temps d'attendre, la piste de Fred est ok, mais il manque la correction pour mobile je pense.
Mis à jour par Nicolas Roche il y a plus d'un an
Juste pour info, j'ai testé ton patch (pour le noyau scss #69398) et oui ça fonctionne tout aussi bien.
L'échéance pour passer l'intégration graphique en production est fixée au 8/11 soit pour nous, avant la mise en prod du 28/10.
Mis à jour par Nicolas Roche il y a plus d'un an
- Lié à Development #70232: toulouse-2022: ticket chapeau des retours sur le nouveau thème ajouté
Mis à jour par Nicolas Roche il y a plus d'un an
- Statut changé de Solution proposée à Rejeté
Rejeté au profit de #69398 qui corrige le comportement du text_indent partout au lieu de contourner juste pour Toulouse.