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

[fr] nth-child rework based on english #20344

Merged
merged 5 commits into from
Oct 10, 2024
Merged

[fr] nth-child rework based on english #20344

merged 5 commits into from
Oct 10, 2024

Conversation

Cajuteq
Copy link
Contributor

@Cajuteq Cajuteq commented May 13, 2024

Description

Reword the :nth-child page based on the english version of the article

Motivation

Issue #16685 pointed a bad wording that could induce errors

Additional details

Instead of just fixing the few words, I re translated the whole paragraph based on the english article

Related issues and pull requests

Fixes #16685

@Cajuteq Cajuteq requested a review from a team as a code owner May 13, 2024 13:53
@Cajuteq Cajuteq requested review from SphinxKnight and removed request for a team May 13, 2024 13:53
@github-actions github-actions bot added the l10n-fr Issues related to French content. label May 13, 2024
Copy link
Contributor

github-actions bot commented May 13, 2024

Preview URLs

Flaws (2)

URL: /fr/docs/Web/CSS/:nth-child
Title: :nth-child
Flaw count: 2

  • macros:
    • Wrong xref macro used (consider changing which macro you use). Error processing path /fr/docs/Web/CSS/:current
  • translation_differences:
    • Differences in the important macros (4 in common of 14 possible)

(comment last updated: 2024-10-10 13:47:17)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Copy link
Contributor

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

@github-actions github-actions bot added the merge conflicts 🚧 This pull request has merge conflicts that must be resolved. label Jul 28, 2024
@github-actions github-actions bot removed the merge conflicts 🚧 This pull request has merge conflicts that must be resolved. label Jul 29, 2024
@@ -16,8 +16,10 @@ body :nth-child(4n) {
}
```

> [!NOTE]
> Pour CSS3, pour que l'élément soit ciblé, il faut qu'il ait un élément parent. En CSS4, cette restriction a été levée.

Copy link
Contributor

Choose a reason for hiding this comment

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

[mdn-linter] reported by reviewdog 🐶

Suggested change

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@SphinxKnight
Copy link
Member

Je présente mes excuses pour l'absence ici @Cajuteq

@SphinxKnight SphinxKnight merged commit 837bd26 into mdn:main Oct 10, 2024
7 checks passed
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

:nth-child - reformulation ?
2 participants