fix: éviter un décalage vertical de la barre d'onglets sur la page de configuration des préférences auteur

auteur (perso) | configurer_langage | configurer_preferences

changed milestone to %prive-2.x

On a un ticket / une cpature d'ecran associé ?

Là ca m'interpelle d'avoir des styles spécifiques à un squelette j'avoue.

Modification anodine il me semblait. Tu peux tester toi-même le décalage vertical en cliquant sur les onglets, c'est subtil, mais assez évident si on y prête attention.

Et oui, je te rejoins sur le fait que cette ligne de CSS dans la balise est un peu incongrue, mais bon... En attendant ... c'est mieux à défaut d'être parfait.

Ah chez moi c'est pas du tout subtil. Mais en fait c'est juste que je trouvais pas très explicite le message de commit: je ne savais pas quoi tester et où.

Cela étant ca m'interpelle vraiment ce truc : a priori le contenu de #chemin c'est un ajax vide dans tous les cas non ? pourquoi ce decalage

(by the way j'aurais mis le commit en fix et pas en chore, cf https://www.spip.net/fr_article6824.html)

(Merci de répondre au commentaiiiiiire, au premier de maieul, c'eut été mieux, et pas dans un nouveau fil différemnt… Sur chaque commentaire racine il y a un bouton flèche avec le titre "Répondre au commentaire", et qui ouvre un éditeur directement collé au thread, et non pas celui tout en bas du bas, qui crée uniquement un nouveau fil.)

C'est mal foutu, c'est mal foutu... Que veux-tu ?! Mea culpa une n-ième fois !

Ah et oui, c'est plutôt fix:en effet.

Proposition de log :

fix: éviter un décalage vertical de la barre d'onglets sur la page de configuration des préférences auteur

Et ça n'est pas chores mais chore cf https://www.spip.net/fr_article6824.html :)

Merci de répondre au commentaiiiiiire

Tout le monde se trompe, tout le temps, toi aussi.
L'UX de Gitlab est pétée.

J'ai créé un style pour bien mettre en évidence le champ "Nouveau fil de commentaires".

Testez ça :
https://userstyles.world/style/20608/git-spip-net-ux

(marche sur thème clair et sombre)

Merci @nicod (ça marche bien d’ailleurs).

Et @rastapopoulos si tu interviens sur la PR juste pour râler sur où est fait la réponse — sans apporter d’information intéressante pour cette PR — c’est absolument pas utile.

Merci de faire des propositions d’amélioration du code / du commit / des relectures… de valider ou non ce qui est proposé… a minima quoi…

added 34 commits

• 94e4e75e...e04b9fe3 - 33 commits from branch spip:2.x
• 0a8b08ce - chores: La position de la barre d'onglets ne varie pas suivant l'onglet sélectionné

Compare with previous version

@placido ? petit up ici ?

@maieul faisait remarquer qu'une correction digne de ce nom déplacerait tout le CSS dans un fichier dédié, plutôt qu'en inline dans le gabarit. Qu'importe selon moi.

Je trouve cette css inline à cet endroit assez mal placée effectivement.
En terme de maintenabilité, c'est pas une bonne pratique.
La CSS c'est peut être pas du vrai code pour vous, mais là c'est mettre la poussière sous le tapis en disant merde à celui/celle qui prendra la suite.

La CSS c'est peut être pas du vrai code pour vous

Je me demande bien où tu es allé chercher cette pointe dans la communauté SPIP.

C’est qui le «pour vous» et qui améliore la PR du coup ou en propose une autre ?

Bon, il semble que le consensus se dessine pour reformuler une PR plus propre (avec CSS dans un fichier idoine).

Donc je vais supprimer cette MR et créer un ticket en bonne et due forme.

• Dans quel fichier donc faudrait il déplacer ce code ?
• Est-ce une bonne chose de cacher ce #chemin ?
• Je vois qu’il semble plutôt partout présent (mais vide) (par exemple sur les stats)
• sauf sur l’auteur en cours, qui a justement caché ça en CSS inline déjà : https://git.spip.net/spip/prive/-/blob/2.x/squelettes/top/auteur.html?ref_type=heads#L3
• introduit il y a 7 ans par 8135be63

Pour pouvoir déplacer ce dernier code dans un fichier CSS ça veut dire s’appuyer sur des attributs qui comparent l’id auteur en cours… dans le body, on a juste des classes accueil statut_0minirezo webmestre_oui auteur_1 spip-theme-colors-11 ce qui va être insuffisant probablement. Ça voudrait dire qu’il faudrait ajouter des attributs genre data-auteur="1" sur le body, et à comparer avec un autre à ajouter qq part pour cacher #chemin en CSS si c’est l’auteur du même numéro. C’est tout de suite une modification plus conséquente.

Quelqu’un veut prendre le temps de le faire ?

---

Stats:

Le ticket dédié : #63 (closed)

En fait je suis même pas sûr que ça soit faisable en CSS

Même si on avait cela :

• body.data-session-auteur=1
• body.data-objet=auteur
• body.data-id_objet=1

Je ne pense pas qu’on puisse comparer en CSS que data-session-auteur = data-id_objet ? Donc ça me semble bien compliqué de passer le CSS qui est actuellement inline et teste la session, ailleurs.

changed title from chores: La position de la barre d'onglets ne varie pas suivant l'onglet sélectionné to fix: éviter un décalage vertical de la barre d'onglets sur la page de configuration des préférences auteur

mentioned in issue #63 (closed)

closed