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

Les anciennes balises SPOILER/SECRET ne fonctionnent plus #5026

Closed
GuimDev opened this issue Aug 26, 2018 · 7 comments · Fixed by #5140
Closed

Les anciennes balises SPOILER/SECRET ne fonctionnent plus #5026

GuimDev opened this issue Aug 26, 2018 · 7 comments · Fixed by #5140
Labels
C-Front Concerne l'interface du site S-Régression Corrige un problème sur un composant qui fonctionnait auparavant

Comments

@GuimDev
Copy link

GuimDev commented Aug 26, 2018

Par exemple :

https://zestedesavoir.com/forums/sujet/835/zep-11-interface-de-statistiques-sur-les-tutoriels/?page=12

On a simplement un gros bloque ouvert.

@artragis artragis added the C-Front Concerne l'interface du site label Aug 26, 2018
@gllmc
Copy link
Member

gllmc commented Aug 26, 2018

J'ai l'impression que ça ne concerne que les posts avec l'ancien zmarkdown (ce qui ne rend pas le bug moins problématique), merci du rapport !

@gllmc gllmc added the S-Régression Corrige un problème sur un composant qui fonctionnait auparavant label Aug 26, 2018
@artragis
Copy link
Member

Ce sont sûrement les classes CSS qui ne sont plus gérées.

@artragis artragis reopened this Aug 26, 2018
@GuimDev
Copy link
Author

GuimDev commented Aug 26, 2018

artragis si tu regardes le html de la page, il manque la balise au dessus du bloc spoiler. Je ne pense pas que le CSS soit généré en HTML donc sûrement du javascript.

<a class="spoiler-title ico-after view" href="#">Afficher/Masquer le contenu masqué</a>

@artragis
Copy link
Member

artragis si tu regardes le html de la page, il manque la balise au dessus du bloc spoiler. Je ne pense pas que le CSS soit généré en HTML donc sûrement du javascript.

le javascript réagit à partir d'un sélecteur du style $('.class-du-spoiler'). Les classes ayant changé avec zmarkdown, il se peut qu'on ait mal porté le truc.

@GuimDev
Copy link
Author

GuimDev commented Aug 27, 2018

@Situphen
Copy link
Member

Oups, my bad :/ J'ai bien pensé à garder l'ancien CSS mais j'ai remplacé bêtement l'ancien JS... Désolé :/

@A-312
Copy link
Contributor

A-312 commented Dec 9, 2018

Ancien HTML :

<div itemprop="text">
    <div class="spoiler">
        <div>
            <p>Coucou</p>
        </div>
    </div>
</div>

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 S-Régression Corrige un problème sur un composant qui fonctionnait auparavant
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants