Projet

Général

Profil

Bug #48101

scroll horizontal sur la page de paramétrage des SMS

Ajouté par Frédéric Péters il y a plus de 3 ans. Mis à jour il y a environ 3 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
28 octobre 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

Parce que champ URL il est mis à 100% de largeur via gadjo et ça passe pas.

De manière générale le style de la page n'est pas correcte parce qu'il y a dans le balisage un <ul></ul> devant le <form>.

(<ul> vide parce que là devrait s'afficher le crédit restant mais c'est quelque chose qui avec le temps a été perdu).


Demandes liées

Lié à w.c.s. - Development #39088: supprimer le code natif d'envoi de SMSFermé19 janvier 2020

Actions

Historique

#1

Mis à jour par Nicolas Roche il y a plus de 3 ans

Peut-être faire en sorte que le style gadjo s'applique au formulaire même si une balise (<ul>) est introduite entre lui et la balise h2 ?
(J'avoue ne pas mesurer les conséquences sur l'existant).

diff --git a/gadjo/static/css/_hacks.scss b/gadjo/static/css/_hacks.scss
index 28e48c0..98c88fa 100644
--- a/gadjo/static/css/_hacks.scss
+++ b/gadjo/static/css/_hacks.scss
@@ -10,3 +10,3 @@ div#appbar + form,  // misc
 div#appbar + ul.messages + form,  // misc
-div#main-content > h2:first-child + form,  // w.c.s.
+div#main-content > h2:first-child ~ form,  // w.c.s.
 div.form-validation form, // w.c.s.
#2

Mis à jour par Frédéric Péters il y a plus de 3 ans

Non, ne pas bricoler sur quelque chose qui est mauvais à la base; ici, dans l'ordre de la démolition il y aurait 1/ virer le <ul></ul> et la gestion de crédits qui de toute façon n'est pas exploitée, 2/ virer les connecteurs SMS != passerelle.

Mais de manière plus immédiatement pratique il y aurait à juste ne pas afficher cette page vu que la configuration se fait via hobo désormais. (i.e. ajouter sms à settings-disabled-screens dans le site-options.cfg des tenants).

#3

Mis à jour par Frédéric Péters il y a environ 3 ans

  • Statut changé de Nouveau à Fermé

Voilà toute la partie SMS réduite à Passerelle, #39088.

#4

Mis à jour par Frédéric Péters il y a environ 3 ans

Formats disponibles : Atom PDF