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

Améliorer la mise en page durant la Markdownisation #15

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

tianyikillua
Copy link
Contributor

@tianyikillua tianyikillua commented Dec 25, 2016

  • Sauts de ligne désormais pilotés uniquement par <p> et <br>, et
    non plus par des \n existants (ce qui est le cas dans
    legifrance il me semble) (présence de \n bidon dans certains
    XML)
  • Des tags tels que <p>, <br>, etc automatiquement supprimés
    par contenu.text
  • Assurer que l'on aura uniquement deux \n entre paragraphs, y
    compris pour les listes (plus lisible pour moi)

- Sauts de ligne désormais pilotés uniquement par <p> et <br>, et
  non plus par des \n existants (ce qui est le cas dans
  legifrance il me semble) (présence de \n bidon dans certains
  XML)
- Des tags tels que <p>, <br>, etc automatiquement supprimés
  par `contenu.text`
- Assurer que l'on aura uniquement deux \n entre paragraphs, y
  compris pour les listes (plus lisible pour moi)
@tianyikillua
Copy link
Contributor Author

tianyikillua commented Jan 2, 2017

Juste pour illustrer ce pull-request, voici le fichier sans #15 ni #17 :

  • les listes numérotées ne sont pas toutes correctement affichées (idem dans le cpi), cf. l'article L111-2 du ceseda
  • on aura des sauts de ligne non désirés (il faut ainsi piloter les sauts uniquement par <p> et <br>), cf. par exemple l'article L214-8 généré par le XML suivant
<CONTENU>
<p>Les articles L. 214-1, L. 214-2, L. 214-3, 
L. 214-5 et L. 214-6 sont applicables sur l'ensemble du territoire de la République. </p>
<p>Au sens des dispositions des articles L. 214-1 à L. 214-3, les expressions : " en France " et " territoire national " s'entendent de l'ensemble du territoire de la République.</p>
</CONTENU>

@tianyikillua tianyikillua mentioned this pull request Jan 2, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant