Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Trop d'espace sur la balise spoiler ? #4457

Closed
A-312 opened this issue Aug 5, 2017 · 6 comments · Fixed by #5140
Closed

Trop d'espace sur la balise spoiler ? #4457

A-312 opened this issue Aug 5, 2017 · 6 comments · Fixed by #5140
Labels
C-Front Concerne l'interface du site Facile Bon ticket pour débuter pour rejoindre le développement !

Comments

@A-312
Copy link
Contributor

A-312 commented Aug 5, 2017

Bonjour,

J'ai l'impression qu'il y a trop d'espace entre le .spoiler-title et le .spoiler, la règle margin-top#L194 n'est pas prise en compte. S'agit-il d'une régression dû à une factorisation du CSS ?

Ce sont les deux premières propriétés qui ne sont pas prise en compte.

.spoiler {
  margin-top: 0;
  padding-left: 15px;
  background: #EEE;
}

A-312.

Il faudrait ajouter un !important au margin-top et padding-left ou déplacer la déclaration de la règle.

@A-312
Copy link
Contributor Author

A-312 commented Aug 5, 2017

image

@motet-a
Copy link
Contributor

motet-a commented Aug 5, 2017

Le sélecteur utilisé pour appliquer padding: 7px 15px 7px 45px; est hyper (trop ?) fort. À mon avis, il vaudrait mieux essayer de le réduire plutôt que de mettre un !important.

@A-312
Copy link
Contributor Author

A-312 commented Aug 9, 2017

La règle que j'ai cité est présente dans le CSS, je suppose qu'a cause d'une factorisation du CSS, cette règle a été déplacé et n'est plus prise en compte dans le rendu.

@Situphen Situphen added the C-Front Concerne l'interface du site label Aug 9, 2017
@A-312
Copy link
Contributor Author

A-312 commented Aug 12, 2017

Je pensais le faire. Mais après 3 essais je n'ai pas reussi (même en annulant mes modifs avec git checkout) a compilé le CSS pour tester le rendu. La commande gulp watch n'a pas arrêté de crier qu'il y avait une erreur avec une variable non déclaré (font machin au début du fichier), même en essayant d'ajouter un simple espace à la fin d'une ligne.

@motet-a
Copy link
Contributor

motet-a commented Aug 12, 2017

C'est étrange. Que t'affiches git status ?

@A-312
Copy link
Contributor Author

A-312 commented Sep 3, 2018

Je prends, ça sera fait demain

@gustavi gustavi added the Facile Bon ticket pour débuter pour rejoindre le développement ! label Sep 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Front Concerne l'interface du site Facile Bon ticket pour débuter pour rejoindre le développement !
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants