Projet

Général

Profil

Bug #20751

thème minint cassé après #16161

Ajouté par Thomas Noël il y a plus de 6 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
15 décembre 2017
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Par exemple avant on avait un display:none pour steps, maintenant y'a plus.

Avant = http://git.entrouvert.org/publik-base-theme.git/tree/static/minint?id=a042c45d829b68ac4683590f57a0fb0324a9b6ab
A comparer avec http://git.entrouvert.org/publik-base-theme.git/tree/static/minint

J'ai l'impression que le style.scss a été juste effacé...?


Fichiers

Révisions associées

Révision cde26571 (diff)
Ajouté par Frédéric Péters il y a plus de 6 ans

minint: restore erroneously removed base css file (#20751)

Under a different name so it's clear it's not generated from sass.

Révision 090be815 (diff)
Ajouté par Frédéric Péters il y a plus de 6 ans

minint: adjust style of textarea and select (#20751)

Historique

#2

Mis à jour par Thomas Noël il y a plus de 6 ans

A noter que dans le theme.html variante minint on fait référence à deux css :
  • static/minint/style.css (qui n'existe plus, donc)
  • static/minint/css/style.css

C'est le premier qui a été supprimé par #16161

Cf en prod sur https://contacts-demarches.interieur.gouv.fr/saisine-par-voie-electronique/sve-ddi/, deux css :
#3

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

Ce qu'il manque c'est uniquement de cacher les étapes du formulaire ou il peut y avoir d'autres petits détails?

#4

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

Ce qu'il manque c'est uniquement de cacher les étapes du formulaire ou il peut y avoir d'autres petits détails?

Mille autre choses à découvrir en regardant attentivement ce qui a été supprimé.

#5

Mis à jour par Thomas Noël il y a plus de 6 ans

Y'a clairement pas mal de trucs :

Je vais installer https://minint.dev.entrouvert.org/ pour pouvoir jouer local.

#6

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

static/minint/style.css (qui n'existe plus, donc)

Et c'était bien là l'erreur; ici je propose simplement de restaurer ce fichier, en lui donnant un nom différent pour éviter les confusions.

#7

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

Tant qu'à faire aussi, mise à jour du style pour les <select> et <textarea>, qu'ils ressemblent aux <input>.

#8

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

Je propose de virer la ligne

/*# sourceMappingURL=style.css.map */

pour éviter toute confusion pour les personnes qui peut-être interviendront sur ce fichier et Chromium/Chrome qui se base dessus pointer les sources SCSS qui ont servi à construire ce fichier.

#10

Mis à jour par Thomas Noël il y a plus de 6 ans

Ack

#11

Mis à jour par Thomas Noël il y a plus de 6 ans

  • Statut changé de En cours à Résolu (à déployer)
commit cde265713f3352b7b1fa8b85d2216c28125480d2
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Mon Dec 18 10:43:39 2017 +0100

    minint: restore erroneously removed base css file (#20751)

    Under a different name so it's clear it's not generated from sass.

#12

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

J'ai encore 0001-minint-adjust-style-of-textarea-and-select-20751.patch.

#13

Mis à jour par Thomas Noël il y a plus de 6 ans

Manque d'attention de ma part, mes excuses.

commit 090be815875f8d8206798dd39e0cbbed28dbf1d4
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Mon Dec 18 10:53:07 2017 +0100

    minint: adjust style of textarea and select (#20751)

#14

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