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

Transformer les balises SPOILER en pure HTML avec SUMMARY et DETAILS #5454

Closed
A-312 opened this issue Sep 29, 2019 · 7 comments
Closed

Transformer les balises SPOILER en pure HTML avec SUMMARY et DETAILS #5454

A-312 opened this issue Sep 29, 2019 · 7 comments
Labels
C-Front Concerne l'interface du site S-Évolution Ajoute de nouvelles fonctionnalités

Comments

@A-312
Copy link
Contributor

A-312 commented Sep 29, 2019

https://www.alsacreations.com/article/lire/1335-html5-details-summary.html

Il faudrait avoir :

<details><summary>TITRE</summary>
<div>
<p>CONTENU</p>
</div>
</details>
TITRE

CONTENU

RAISON : On aurait plus besoin de l'event "onclick"

Attention lors du Q/A :

2ème étape :

  • Modifier ZMD pour qu'il utilise directement SUMMARY et DETAILS au lieu du format en div (= 2ème break change).
@A-312 A-312 added the C-Front Concerne l'interface du site label Sep 29, 2019
@artragis
Copy link
Member

il suffit d'activer un flag dans zmd, ensuite si on gère bien les choses c'est qu'une question de classe CSS.

Par contre il faut absolument mieux styliser la chose car la flèche elle est moche.

@Situphen Situphen added the S-Évolution Ajoute de nouvelles fonctionnalités label Sep 29, 2019
@AmauryCarrade
Copy link
Member

Attention que <détails> n'est pas supporté par Edge (et IE 11 mais je ne crois pas qu'on le supporte de toute façon). Il faudra peut-être un fallback.

@A-312
Copy link
Contributor Author

A-312 commented Sep 29, 2019

Ah EDGE... :-/

@artragis
Copy link
Member

edge est aujourd'hui sur chromium donc je dirais que c'est supporté. Tu as une source @AmauryCarrade ?

@A-312
Copy link
Contributor Author

A-312 commented Sep 29, 2019

Ne fonctionne pas sous Edge 44

@AmauryCarrade
Copy link
Member

AmauryCarrade commented Sep 29, 2019 via email

@artragis artragis reopened this Oct 13, 2019
@Situphen
Copy link
Member

Mise en place lors du passage à zmarkdown 11 avec #6276

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-Évolution Ajoute de nouvelles fonctionnalités
Projects
None yet
Development

No branches or pull requests

4 participants