Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Fix #972 sur les doubles espaces dans contributing#1113

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
JulienPalard merged 3 commits into3.8fromcontributing-doubles-espaces
Feb 5, 2020

Conversation

Seluj78
Copy link
Collaborator

No description provided.

au HTML et au PDF, qui ne tiennent pas compte du double-espace.
Nous avions décidé de ne rien changer pour les double-espaces
coté traduction : nous ne les retirons pas et ce n'est pas grave
si les traducteurs les retirent par accident

Choose a reason for hiding this comment

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

vpoulailleau and JulienPalard reacted with thumbs up emoji
Copy link
Contributor

Choose a reason for hiding this comment

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

Nous sommes sur une question de typographie. Donc, ce qui compte c'est le résultat une fois rendu dans le format final de lecture.
padpo utiliserst2txt qui maintient les doubles espaces, ce qui fait que Grammalecte génère une erreur (pas un avertissement).
rst2html conserve également les doubles espaces dans la sortie HTML, mais ceux-ci sont ensuite ignorés par le rendu HTML (conformément à la norme).
Enfin, la détection des « espaces surnuméraires » est une option de Grammalecte (et de grammalecte-cli).
En conclusion et comme je considère que le format final important est HTML, je propose de désactiver l'option « espaces surnuméraires » de Grammalecte et de ne pas corriger les doubles-espaces déjà présent.

JulienPalard reacted with thumbs up emoji
Copy link
Contributor

Choose a reason for hiding this comment

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

@christopheNan Les doubles espaces remontés par Grammalecte sont ignorés par padpo, mais un autre checker de padpo (plus ancien) détecte les doubles espaces directement dans le RST (https://github.com/vpoulailleau/padpo/blob/master/padpo/checkers/doublespace.py). Celui-ci peut être supprimé, dès que le consensus est atteint.

J'aimais bien le commentaire de@awecx (#972 (comment)), et padpo ne signale, sur analyse d'une pull request, que les messages concernant les items modifiés, ce qui est en phase avec le commentaire mentionné.

De mon côté, je suis d'accord avec@christopheNan que le plus important est le rendu final de lecture qui doit être « parfait ». Mais j'avoue que la proposition de@awecx me semble encore plus pertinente.

padpo sera donc à modifier en fonction de la décision de cette PR

PS : padpo n'utilise pas rst2txt et rst2html, mais analyse directement les chaînes de caractères (sauf pour la partie Grammalecte, ou je génère un texte adapté à Grammalecte pour avoir des messages d'erreur pertinents)

Copy link
CollaboratorAuthor

Choose a reason for hiding this comment

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

Ping, que décidons nous ?

Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
Co-Authored-By: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
@JulienPalardJulienPalard merged commite226a1a into3.8Feb 5, 2020
@Seluj78Seluj78 deleted the contributing-doubles-espaces branchDecember 3, 2020 10:23
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@vpoulailleauvpoulailleauvpoulailleau left review comments

@christopheNanchristopheNanchristopheNan left review comments

@awecxawecxawecx left review comments

@JulienPalardJulienPalardJulienPalard approved these changes

@deronnaxderonnaxAwaiting requested review from deronnax

Labels
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

5 participants
@Seluj78@JulienPalard@vpoulailleau@christopheNan@awecx

[8]ページ先頭

©2009-2025 Movatter.jp