Projet

Général

Profil

Bug #7760

Le chargement de jquery-ui.min.css rend difficile de thémer les formulaires

Ajouté par Benjamin Dauvergne il y a presque 9 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
03 juillet 2015
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

À voir sur https://cresson.entrouvert.org/manage/users/ en essayant d'ajouter un utilisateur.

(Le problème est surtout visible après suppressions de la règle problèmatique de #7757)

jquery-ui.min.css déclare cette règle:

.ui-widget input, .ui-widget select, .ui-widget textarea, .ui-widget button {
    font-family: Verdana,Arial,sans-serif;
    font-size: 1em;
}

qui change la hauteur de l'élément <select> de choix d'une entité, et plus rien n'est aligné. Si je bloque ces deux déclaration c'est bon. Plus généralement je me dis qu'on pourrait nous même définir le style des boites de dialogue et ne pas utiliser un thème jQuery en plus de gadjo.


Fichiers

0001-remove-dependency-on-jquery-ui-css-7760.patch (3,76 ko) 0001-remove-dependency-on-jquery-ui-css-7760.patch Frédéric Péters, 20 novembre 2016 15:40
foo.png (13,3 ko) foo.png Frédéric Péters, 22 novembre 2016 15:43
0001-remove-dependency-on-jquery-ui-css-7760.patch (6,75 ko) 0001-remove-dependency-on-jquery-ui-css-7760.patch Frédéric Péters, 01 janvier 2017 15:17

Demandes liées

Lié à w.c.s. - Development #14466: retirer le chargement des CSS de jquery uiFermé01 janvier 2017

Actions

Révisions associées

Révision 29f7e00f (diff)
Ajouté par Frédéric Péters il y a environ 7 ans

remove dependency on jquery ui css (#7760)

This adds required styles for dialogs, overlays and calendar widget.

Historique

#1

Mis à jour par Benjamin Dauvergne il y a presque 9 ans

  • Description mis à jour (diff)
#2

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

#3

Mis à jour par Serghei Mihai il y a plus de 7 ans

Tu peux rajouter une capture d'écran?
Chez moi, dans le dialog d'ajout d'utiliser le select de l'entité est bien aligné, avec FF et Chromium.

#4

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

Tu parles du bug d'il y a plus d'un an de Benjamin ? Non, je ne pense pas que tu obtiennes une capture d'écran.

#5

Mis à jour par Serghei Mihai il y a plus de 7 ans

Si j'ai bien compris, le souci doit être visibile sur cresson, lors de l'ajout d'un utilisateur.
Sur ma machine le select est bien aligné.

#6

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

C"était il y a plus d'un an, un détail de comportement comme ça a bien des choses d'avoir changé, sans heurter le fond, qui est que certaines règles amenées par jquery-ui.css rendent les choses compliquées.

Aussi, Verdana, Arial, sans-serif; si en local tu n'as ni Verdana ni Arial, le fallback sans-serif va jouer et la même police se trouvera du coup partout, par chance.

Bref, pas de capture, l'exemple donnée était juste un exemple.

#7

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

Mais une capture pour montrer que le nouveau style est tout à fait similaire à l'ancien. (dans le jeu des sept erreurs, il y a la croix de fermeture, qui était avant une image, qui est désormais une icône de font awesome).

#8

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

Patch refait en déplaçant tout ce qui relève de jquery ui dans un fichier spécifique (ça permettra plus facilement de l'avoir également dans publik-base-theme).

#9

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

#10

Mis à jour par Serghei Mihai il y a environ 7 ans

Ack

#11

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

  • Statut changé de En cours à Résolu (à déployer)
commit 29f7e00f032a4b7bd5a302f3e9f8831a3ad91f8e
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Sun Nov 20 15:38:33 2016 +0100

    remove dependency on jquery ui css (#7760)

    This adds required styles for dialogs, overlays and calendar widget.
#12

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

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF