Skip to content

Update index.md / Erreur de guillemets autour d'une valeur XML-invalide #25318

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

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

PhilippePerret
Copy link

Description

Typo sur exemple.

Motivation

Corriger l'exemple donné pour qu'il ait un sens.

@PhilippePerret PhilippePerret requested a review from a team as a code owner January 3, 2025 14:52
@PhilippePerret PhilippePerret requested review from SphinxKnight and removed request for a team January 3, 2025 14:52
@github-actions github-actions bot added the l10n-fr Issues related to French content. label Jan 3, 2025
Copy link
Contributor

github-actions bot commented Jan 3, 2025

Preview URLs

Flaws (36)

URL: /fr/docs/Web/HTML/Attributes
Title: Référence des attributs HTML
Flaw count: 36

  • macros:
    • /fr/docs/Web/HTML/Attributes/dirname does not exist but fell back to /en-US/docs/Web/HTML/Attributes/dirname
    • /fr/docs/Web/HTML/Attributes/placeholder does not exist but fell back to /en-US/docs/Web/HTML/Attributes/placeholder
  • broken_links:
    • Can't resolve /fr/docs/Web/HTML/Attributes/alt
    • Can't resolve /fr/docs/Web/HTML/Attributes/autoplay
    • Can't resolve /fr/docs/Web/HTML/Attributes/cite
    • Can't resolve /fr/docs/Web/HTML/Attributes/colspan
    • Can't resolve /fr/docs/Web/HTML/Attributes/controls
    • and 29 more flaws omitted
External URLs (1)

URL: /fr/docs/Web/HTML/Attributes
Title: Référence des attributs HTML

(comment last updated: 2025-01-04 00:38:42)

Copy link
Member

@SphinxKnight SphinxKnight left a comment

Choose a reason for hiding this comment

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

La documentation explique justement ce cas.
En anglais (la version de référence en cas de doute) il y a bien cet exemple : https://developer.mozilla.org/en-US/docs/Web/HTML/Attributes#boolean_attributes

On a ici un attribut booléen qu'on indique comme "vrai" en fournissant sa valeur.
Bien que cette notation soit très peu conventionnelle, elle est documentée ici.

Je suis d'avis de laisser la page telle quelle quant à ce point.

Si vous êtes d'accord, nous fermerons la PR. Sinon, discutons-en :)

@PhilippePerret
Copy link
Author

Bonjour SphinxKnight,

Non, la page en anglais ne présente pas d'erreur, contrairement à la page en français, que je t'invite à lire attentivement.

Dans la page en français, il y a deux fois la même chose présenté avec une fois indiqué "code XML invalide" et l'autre fois, avec exactement la même chose, "code XML valide". Tout simplement parce que des guillemets ont été ajoutés par le traducteur.

Il faudrait donc trouver :

< .... itemscope=itemscope>
code XML invalide [et pour cause <= il manque les guillemets]

au lieu de :

< .... itemscope="itemscope">
… code XML invalide [ce qui est absurde, puisque c'est en réalité du code XML valide]

On trouve juste en dessous le bon code :

< .... itemscope="itemscope">
… code XML valide [et là c'est juste]

J'espère que c'est plus clair à présent.

Bien à toi.

@tristantheb
Copy link
Member

Je vois le problème énoncé et en image ce sera plus parlant :
image

Ce qui fait que actuellement l'exemple 2 et 4 se contre-disent textuellement en français oui (@SphinxKnight si tu vois de quoi ça parle maintenant, c'est juste le linter qui est désactivé sur la version US et qui ne l'est pas chez nous)

* remove deprecated element from en-US
* add missing content from en-US
* fix translations
Copy link
Contributor

Preview URLs

External URLs (1)

URL: /fr/docs/Web/HTML/Attributes
Title: Référence des attributs HTML

@github-actions github-actions bot added the merge conflicts 🚧 This pull request has merge conflicts that must be resolved. label Jul 4, 2025
Copy link
Contributor

github-actions bot commented Jul 4, 2025

This pull request has merge conflicts that must be resolved before it can be merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
l10n-fr Issues related to French content. merge conflicts 🚧 This pull request has merge conflicts that must be resolved.
Projects
Status: 👀 Review requested
Development

Successfully merging this pull request may close these issues.

3 participants