Eviter que les titres longs dépassent. Fix #5284 #5294

Merged
b_b merged 1 commits from fix_5284 into master 1 week ago
Owner

Pour corriger le bug d'affichage des titres longs #5284

Pour corriger le bug d'affichage des titres longs #5284
erational added 1 commit 1 month ago
tcharlss requested changes 3 weeks ago
color: var(--spip-color-theme-dark);
font-weight: normal;
overflow-wrap: break-word;
max-width: calc(100% - 75px);
Owner

Désolé, pinaillage du vendredi :)

Ici plutôt que mettre 75px à vue de nez, on pourrait reprendre exactement la largeur définie pour l'icône + une petite marge.

Actuellement c'est 5em :

.icone a {
    width: 5em;
}

Mais pour être sûr d'avoir tout le temps la même valeur, je propose d'ajouter une variable :

:root {
    --spip-btnicon-width: 5em;
}
.icone a {
    width: var(--spip-btnicon-width);
}
#contenu .fiche_objet > .box__header .soustitre {
    calc(100% - (--var(--spip-btnicon-width) + 1em));
}

Nb : pour le nom des variables, en général j'essaye de faire --spip-<module>-<propriété>.
Là le module ça peut pas être pas icon tout court (trop générique), donc j'ai repris ce qu'il y a dans la charte ou c'est appelé des « boutons icônes ». Faute de mieux.

Désolé, pinaillage du vendredi :) Ici plutôt que mettre 75px à vue de nez, on pourrait reprendre exactement la largeur définie pour l'icône + une petite marge. Actuellement c'est [5em](https://git.spip.net/spip/spip/src/branch/master/prive/themes/spip/icons.css.html#L57) : ``` .icone a { width: 5em; } ``` Mais pour être sûr d'avoir tout le temps la même valeur, je propose d'ajouter une variable : ```css :root { --spip-btnicon-width: 5em; } .icone a { width: var(--spip-btnicon-width); } ``` ```css #contenu .fiche_objet > .box__header .soustitre { calc(100% - (--var(--spip-btnicon-width) + 1em)); } ``` Nb : pour le nom des variables, en général j'essaye de faire `--spip-<module>-<propriété>`. Là le module ça peut pas être pas `icon` tout court (trop générique), donc j'ai repris ce qu'il y a dans la charte ou c'est appelé des « boutons icônes ». Faute de mieux.
Owner

C'est fait en 9f1a4be45e

(il y a un peu plus dans le commit, j'en ai profité pour passer en variables d'autres trucs).

C'est fait en 9f1a4be45e (il y a un peu plus dans le commit, j'en ai profité pour passer en variables d'autres trucs).
tcharlss marked this conversation as resolved
tcharlss added 1 commit 2 weeks ago
tcharlss added 1 commit 2 weeks ago
tcharlss approved these changes 2 weeks ago
b_b commented 2 weeks ago
Owner

Super, quelq'un peu squasher tout ça pour ne garder que le dernier commit ?

Super, quelq'un peu squasher tout ça pour ne garder que le dernier commit ?
marcimat approved these changes 2 weeks ago
erational added 2 commits 1 week ago
Poster
Owner

Super, quelq'un peu squasher tout ça pour ne garder que le dernier commit ?

j'ai squashé mon commit avec celui de tcharlss mais ca a pris en mm temps le changelog :/

> Super, quelq'un peu squasher tout ça pour ne garder que le dernier commit ? j'ai squashé mon commit avec celui de tcharlss mais ca a pris en mm temps le changelog :/
b_b commented 1 week ago
Owner

Super, quelq'un peu squasher tout ça pour ne garder que le dernier commit ?

j'ai squashé mon commit avec celui de tcharlss mais ca a pris en mm temps le changelog :/

Non non, ça me semble tout bon mignon :)

#5294/commits

> > Super, quelq'un peu squasher tout ça pour ne garder que le dernier commit ? > > j'ai squashé mon commit avec celui de tcharlss mais ca a pris en mm temps le changelog :/ Non non, ça me semble tout bon mignon :) https://git.spip.net/spip/spip/pulls/5294/commits
b_b added 2 commits 1 week ago
b_b commented 1 week ago
Owner

Ha non c'est pas tout bon du tout en fait 390986dd64 :p

Ha non c'est pas tout bon du tout en fait https://git.spip.net/spip/spip/commit/390986dd645a97e995cb5411f077e4fe3e05cdec :p
b_b added 1 commit 1 week ago
b_b merged commit dcb3a9cf31 into master 1 week ago
b_b deleted branch fix_5284 1 week ago

Reviewers

tcharlss approved these changes 2 weeks ago
marcimat approved these changes 2 weeks ago
The pull request has been merged as dcb3a9cf31.
Sign in to join this conversation.
Loading…
There is no content yet.