Bug #32680
closedpour les très écrans très larges, décaler encore plus le titre
0%
Description
On fait souvent du "text-indent: -1000px;" mais sur des écrans de foufou comme le mien, ce n'est pas assez, cf copie d'écran.
Files
Updated by Thomas Noël almost 7 years ago
A priori il faut passer tout ça à -9999px, comme fait ailleurs :
static/amiens/_custom.scss: text-indent: -2000px; static/bron/_custom.scss: text-indent: -2000px; static/clisson-sevre-et-maine-agglo/_custom.scss: text-indent: -2000px; static/clisson/_custom.scss: text-indent: -2000px; static/dardilly/_custom.scss: text-indent: -2000px; static/fontenay-sous-bois-2017/_custom.scss: text-indent: -2000px; static/gorges/_custom.scss: text-indent: -2000px; static/grandlyon/_custom.scss: text-indent: -1000px; static/grandlyon/_custom.scss: text-indent: -1000px; static/groupe-up/_custom.scss: text-indent: -2000px; static/haute-goulaine/_custom.scss: text-indent: -2000px; static/la-reunion/_custom.scss: text-indent: -2000px; static/meudon/_custom.scss: text-indent: -1000px; static/montpellier/_custom.scss: text-indent: -1000px; static/nanterre/_custom.scss: text-indent: -1000px; static/saint-lo-agglo/_custom.scss: text-indent: -3000px; static/seine-et-marne-apa/_custom.scss: text-indent: -2000px; static/seine-et-marne-apa/_custom.scss: text-indent: -2000px; static/seine-et-marne-apa/_custom.scss: text-indent: -2000px; static/sicoval/_custom.scss: text-indent: -1000px; static/toulouse/_custom.scss: text-indent: -1000px; static/toulouse/_custom.scss: text-indent: -1000px;
Updated by Thomas Noël almost 7 years ago
- File 0001-use-10000px-indentation-to-hide-texts-32680.patch 0001-use-10000px-indentation-to-hide-texts-32680.patch added
- Tracker changed from Support to Bug
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
Updated by Frédéric Péters (de retour le 10/3) almost 7 years ago
(tant qu'à passer là-dessus ça serait mieux d'assurer un display: (inline-)block; overflow: hidden; qu'il n'y ait tout simplement rien qui ne dépasse).
Updated by Thomas Noël about 4 years ago
- Status changed from Solution proposée to Nouveau
Frédéric Péters a écrit :
(tant qu'à passer là-dessus ça serait mieux d'assurer un display: (inline-)block; overflow: hidden; qu'il n'y ait tout simplement rien qui ne dépasse).
Je n'avais pas compris cette remarque à l'époque, et toujours pas aujourd'hui. Mais le «bogue« est toujours visible sur les écrans wide (+ de 3000px).
Updated by Frédéric Péters (de retour le 10/3) over 3 years ago
- Related to Développement #66815: bethune-bruay, support des titres très longs added
Updated by Frédéric Péters (de retour le 10/3) over 3 years ago
- Status changed from Nouveau to Fermé
Alors, c'était 1/ ajouter overflow: hidden; pour que ce qui sort de la zone allouée pour l'affichage du logo (par exemple un texte mis à -1000px) ne soit pas affiché, mais 2/ #33974 a mis -10000px par défaut.
Mais 3/ c'était il y a 3 ans,
Je n'avais pas compris cette remarque à l'époque, et toujours pas aujourd'hui. Mais le «bogue« est toujours visible sur les écrans wide (+ de 3000px).
Le ticket lié c'est autre chose, bethune-bruay redéfinissait sa limite à -1000px. (mais aussi même à -1000px il y avait quand même le overflow: hidden; et chez moi ça suffit, et le ticket n'a pas de capture d'écran qui illustrerait peut-être autre chose).
Updated by Frédéric Péters (de retour le 10/3) over 3 years ago
- File 0001-misc-always-hide-site-title-text-32680.patch 0001-misc-always-hide-site-title-text-32680.patch added
- Status changed from Fermé to Solution proposée
Mais en fait le patch attaché ne concernait pas le cas générique mais tout un tas de petites variations dans différentes intégrations, pour faire le tour actualisé,
- 1/ j'ignore les -9999px, inutile de se poser des questions là pour peut-être changer la valeur d'un seul pixel,
- 2/ ça laisse des -2000px et des -1000px,
- 3/ certains sont limités au mobile, exemple clisson, donc pas concernés par le risque "écran très large",
@media screen and ($max-mobile-viewport) {
width: 100%;
padding-left: 0;
overflow: hidden;
text-indent: -2000px;
}
- 4/ pour groupe-up c'est pas le logo mais le lien déconnexion, comme c'est un thème démo/pas utilisé, je passe,
- 5/ pour un tas, on a text-indent: -1000px et overflow: hidden et quand je regarde ça m'a l'air tout à fait ok, donc je laisse,
- 6/ concernés, seine-et-marne-apa, grandlyon, montpellier, nanterre, patch attaché. (j'y fait le combo -10000px + overflow: hidden, bien que pour moi le -10000px ne soit pas nécessaire).
Concernant l'effet du overflow: hidden, il faut par exemple regarder https://portail-castelnau-le-lez.eservices.montpellier3m.fr/
Updated by Thomas Noël over 3 years ago
- Status changed from Solution proposée to Solution validée
Updated by Frédéric Péters (de retour le 10/3) over 3 years ago
- Status changed from Solution validée to Résolu (à déployer)
commit dea7ec767e01819d04154131361315df39761afb
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Thu Jul 14 23:01:13 2022 +0200
misc: always hide site title text (#32680)
Updated by Transition automatique over 3 years ago
- Status changed from Résolu (à déployer) to Solution déployée