Skip to content
This repository has been archived by the owner on May 10, 2020. It is now read-only.

Nouvelle extension pour la typographie française. #102

Closed

Conversation

DominusCarnufex
Copy link
Contributor

@DominusCarnufex DominusCarnufex commented Jul 4, 2016

Il s’agit d’une reprise de la PR #76 concernant le nouveau module de typographie française. Par rapport à la PR précédente, voici les changements.

  • J’ai intégré la suggestion de @davbaumgartner d’utiliser   plutôt que   pour les espaces insécables fines.
  • J’ai corrigé un bogue qui faisait que les espaces insécables n’étaient pas insérées devant : et ; si ceux-ci terminaient la ligne.
  • J’ai mis à jour les fichiers de tests pour que les résultats prennent en compte les modifs faites par ce module. En revanche, je n’arrive pas à utiliser le système de test actuellement en usage, donc je n’ai pas pu rédiger de test spécifique à cette extension.
  • Il est apparu un bogue que je ne parviens pas à corriger : le smiley :'( n’est plus reconnu. En effet, l’apostrophe devient une apostrophe typographique, et lorsque je modifie l’expression régulière pour qu’elle ne prenne pas en compte la suite :'(, alors, c’est l’extension qui gère les smileys qui ne le détecte plus. Je suis un peu coincé, là… Apparemment, ça venait de moi, TRAVIS n’a pas le problème.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 93.999% when pulling 3cc3d06 on DominusCarnufex:typographie into 0fd12b1 on zestedesavoir:master-zds.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 93.999% when pulling 3cc3d06 on DominusCarnufex:typographie into 0fd12b1 on zestedesavoir:master-zds.

@cgabard
Copy link
Contributor

cgabard commented Jul 4, 2016

Il est apparu un bogue que je ne parviens pas à corriger : le smiley :'( n’est plus reconnu.

Ton extension devrait passer après celle des émoticons, donc ça devrait fonctionner. Je vais regarder quand j'aurais un peu de temps

@coveralls
Copy link

coveralls commented Jul 4, 2016

Coverage Status

Coverage increased (+0.2%) to 94.08% when pulling 10887ff on DominusCarnufex:typographie into 0fd12b1 on zestedesavoir:master-zds.

@DominusCarnufex
Copy link
Contributor Author

Ouais, ça doit être lié à ma manière de faire les tests, j’ai d’autres bogues bizarres qui apparaissent comme ça. En tout cas, avec les derniers ajouts, TRAVIS passe à merveille, donc tout a l’air bon (à part l’absence de tests spécifiques au module).

b' \xe2\x80\xb0'.decode('utf-8'), " ‰", md
)
openingAngleQuoteWithSpacePattern = ReplacePattern(
b'\xc2\xab '.decode('utf-8'), "« ", md
Copy link

@dvbmgr dvbmgr Jul 4, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pourquoi pas insécable ici ? (edit: c'est peut-être aussi un truc de typographie suisse, note faite.)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ce sont des espaces insécables. Mais des espaces-mots, pas des espaces fines. :-)

Tout est (dernière section secrète, tout en bas).

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oui pardon, je pensais fine. Merci pour les conventions ZdS (;

@DominusCarnufex
Copy link
Contributor Author

Up @cgabard : du nouveau ? Qu’est-ce qui empêche encore le merge ?

@dvbmgr
Copy link

dvbmgr commented Nov 1, 2016

Ping ?

@cgabard
Copy link
Contributor

cgabard commented Nov 2, 2016

Il faut que je termine le chantier de ménage avant (bientôt) et je ferai une passe d'ajout de ces PR.

Il y a eu

@cgabard
Copy link
Contributor

cgabard commented Nov 2, 2016

Désolé, problème de tel...

Bref au passage j'aurais bien aimé que ça soit vite fait commenté sur le site (pas à en faire une zep mais au moins un sujet) juste pour être sûrs qu'il n'y a pas d'objection et de défaut auxquels on aurait pas pensé

@dvbmgr
Copy link

dvbmgr commented Nov 2, 2016

Oui enfin sans vouloir être méchant, ça fait 3 mois qu'il n'y a pas eu de commits ici.

Il y aura des objections parce que des gens vont pas comprendre, et ça va prendre trois mois à être mergé.

Les problèmes que ça peut introduire sont clairs: si quelqu'un s'amuse à mettre du code dans un message sans les balises ça va pas être beau à voir. Si quelqu'un copie du code qui a été écrit dans un message sans balises, ça va pas être beau à voir. Si quelqu'un poste un lien sans mettre la balise de lien, ça va pas être beau à voir.

Mais à un moment, il faut arrêter de prendre les utilisateurs pour des idiots, ou alors qu'ils ne se plaignent pas de l'être.

Et comme discuté, cela force à respecter les conventions typographiques établies sur le site. Il n'y a pas à discuter davantage.

@cgabard
Copy link
Contributor

cgabard commented Nov 2, 2016

Who on s'énerve pas...

Si tu veux reprendre la maintenance et le dev du bouzin et tout ce que ça
implique, vas y.

Ensuite :

Si quelqu'un poste un lien sans mettre la balise de lien, ça va pas être
beau à voir.

Ça je peux pas accepter car le zmarkdown actuellement l'autorise
explicitement donc si ça pète, c'est problématique !

Et comme discuté, cela force à respecter les conventions typographiques
établies sur le site. Il n'y a pas à discuter davantage.

Ça a été discuté ou ça, déjà ? Ensuite je demande pas de faire une zep
hein, juste que ce soit clairement dit quelque part. Sinon je peux aussi
appliquer strictement la règle actuel : "pas de changement de
fonctionnalités tant qu'on a pas tout réorganisé le code et que les pdf
sont nickel" mais je suis pas sûrs que tu préfère cette solution.

Le mer. 2 nov. 2016 10:04, David Baumgartner notifications@github.com a
écrit :

Oui enfin sans vouloir être méchant, ça fait 3 mois qu'il n'y a pas eu de
commits ici.

Il y aura des objections parce que des gens vont pas comprendre, et ça va
prendre trois mois à être mergé.

Les problèmes que ça peut introduire sont clairs: si quelqu'un s'amuse à
mettre du code dans un message sans les balises ça va pas être beau à voir.
Si quelqu'un copie du code qui a été écrit dans un message sans balises, ça
va pas être beau à voir. Si quelqu'un poste un lien sans mettre la balise
de lien, ça va pas être beau à voir.

Mais à un moment, il faut arrêter de prendre les utilisateurs pour des
idiots, ou alors qu'ils ne se plaignent pas de l'être.

Et comme discuté, cela force à respecter les conventions typographiques
établies sur le site. Il n'y a pas à discuter davantage.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#102 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF0jfP-deKpubNCDIeGiD-x3NKHDkMI_ks5q6FIJgaJpZM4JEST1
.

@dvbmgr
Copy link

dvbmgr commented Nov 2, 2016

Je ne m'énerve pas, je fais juste remarquer qu'il n'y a pas eu de commits depuis trois mois et que rien n'avance. Et je trouve ça problématique.

Ça je peux pas accepter car le zmarkdown actuellement l'autorise
explicitement donc si ça pète, c'est problématique !

Alors il manque un test explicite pour ça (probablement qu'il faut compléter tests/zds/rediger_sur_zds_part3.txt), mais cela fonctionne comme attendu.

Ça a été discuté ou ça, déjà ? Ensuite je demande pas de faire une zep

https://zestedesavoir.com/contenus/beta/1027/conventions-typographiques-en-usage-sur-zeste-de-savoir/

fonctionnalités tant qu'on a pas tout réorganisé le code et que les pdf
sont nickel" mais je suis pas sûrs que tu préfère cette solution.

Oui enfin il y a bien des chances que ZMarkdown passe à la trappe pour générer les PDFs à terme. Ça me semble assez suicidaire de tout « refactoriser » (=tout réécrire) alors qu'il manque de dévs et qu'il y a probablement des outils qui le feraient très bien sans nécessiter ce travail.

@cgabard
Copy link
Contributor

cgabard commented Nov 2, 2016

Il n'y en a pas eu sur master car ce sur quoi je travaille quand j'ai le
temps demande plusieurs modifs un peu partout (doit y avoir une branche sur
le dépôt plus récente déjà).

Pour le pdf on va pas refaire le débat ici.


De mon côté je vais finir le nettoyage des extensions et tests (j'arrive à
la fin) et la suite prévu était de merger ces nouvelles fonctionnalités
(celle-ci, le ping et celle de firm1 avec quelques ajustements)

Le mer. 2 nov. 2016 10:56, David Baumgartner notifications@github.com a
écrit :

Je ne m'énerve pas, je fais juste remarquer qu'il n'y a pas eu de commits
depuis trois mois et que rien n'avance. Et je trouve ça problématique.

Ça je peux pas accepter car le zmarkdown actuellement l'autorise
explicitement donc si ça pète, c'est problématique !

Alors il manque un test explicite pour ça (probablement qu'il faut
compléter tests/zds/rediger_sur_zds_part3.txt), mais cela fonctionne
comme attendu.

Ça a été discuté ou ça, déjà ? Ensuite je demande pas de faire une zep

https://zestedesavoir.com/contenus/beta/1027/conventions-typographiques-en-usage-sur-zeste-de-savoir/

fonctionnalités tant qu'on a pas tout réorganisé le code et que les pdf
sont nickel" mais je suis pas sûrs que tu préfère cette solution.

Oui enfin il y a bien des chances que ZMarkdown passe à la trappe pour
générer les PDFs à terme. Ça me semble assez suicidaire de tout
« refactoriser » (=tout réécrire) alors qu'il manque de dévs et qu'il y a
probablement des outils qui le feraient très bien sans nécessiter ce
travail.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#102 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF0jfHEF5p6WfXULmZwWi8RQz471dmzxks5q6F5NgaJpZM4JEST1
.

@GerardPaligot
Copy link
Member

Je viens juste mettre un lien ici entre cette PR ci-présente et cette issue puisque cette PR pourrait bien casser les smileys. C'est à tester.

@DominusCarnufex
Copy link
Contributor Author

Bonsoir les gens.

Alors pour répondre aux remarques et interrogations diverses…

Cette extension typographique, ça fait quasiment un an et demi qu’elle est en discussion sur le forum. La première proposition a été faite dans ce sujet mais sans que ce soit suivi d’effet. Elle a été remise au goût du jour dans ce sujet qui est, pour partie, à l’origine de la ZEP-41. Quant à la définition exacte des règles à respecter, elle est évidemment donnée dans les conventions typographiques.

Elle ne casse pas les liens libres : elle n’opère que si les deux-points et le pour-cent sont précédés d’une espace, ce qui n’est pas le cas dans une URL.

Elle ne casse pas non plus les smileys, j’y ai veillé. Elle agit après l’extension qui gère les smileys, donc pas de problème. Si jamais l’extension des smileys venait à se retrouver après celle de typographie, seul le smiley :'( serait impacté, à cause de l’apostrophe : les deux-points et point-virgules ne sont affectés que s’ils sont suivis d’une espace ou d’une fin de ligne. Ainsi, : D est affecté, mais pas :D.

De manière plus générale, cette extension a été conçue de manière à ne pas interférer avec la typographie des autres langues que le français, donc elle n’affecte que des points qui sont sans aucun doute possible de la ponctuation française. Par exemple, les guillemets droits " ne sont pas transformés automatiquement, parce qu’ils pourraient se trouver dans un texte en anglais ou en allemand ; pour avoir des guillemets français sans les taper soi-même, il faut utiliser << et >>.

J’espère que cela vous rassure sur le soin apporté à cette extension. 😉

@cgabard
Copy link
Contributor

cgabard commented Nov 9, 2016

Si ça te dérange pas, je vais probablement m'occuper du rebase et pour faciliter ça faire ça dans une autre branche/PR

@cgabard
Copy link
Contributor

cgabard commented Nov 9, 2016

Je ferme ici, je merge dans #118
merci @DominusCarnufex

@cgabard cgabard closed this Nov 9, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants