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.

Sign up for GitHub

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

Jump to bottom

fr: remove the {{DOMAttributeMethods}} macros - Fixes #19044 #19081

Merged
merged 2 commits into from Apr 14, 2024
Merged

fr: remove the {{DOMAttributeMethods}} macros - Fixes #19044 #19081

merged 2 commits into from Apr 14, 2024

Conversation

quentin-rey
Copy link
Contributor

Description

fix: remove {{DOMAttributeMethods}} macro

  • add/update Specifications section
  • add/update Compat section
  • add/update See also section
    • listing complementary methods
  • Fixing broken links

Motivation

Participates in resolving #19009

Additional details

Files updated:

  • files/fr/web/api/element/hasattribute/index.md
  • files/fr/web/api/element/setattributens/index.md
  • files/fr/web/api/element/setattributenode/index.md
  • files/fr/web/api/element/getattributenode/index.md
  • files/fr/web/api/element/hasattributens/index.md
  • files/fr/web/api/element/getattributens/index.md
  • files/fr/web/api/element/removeattributenode/index.md
  • files/fr/web/api/element/setattributenodens/index.md
  • files/fr/web/api/element/getattributenodens/index.md
  • files/fr/web/api/element/removeattributens/index.md
  • files/fr/web/api/element/removeattribute/index.md
  • files/fr/web/api/element/setattribute/index.md

Related issues and pull requests

Fixes #19044

@quentin-rey quentin-rey requested a review from a team as a code owner March 29, 2024 21:55
@quentin-rey quentin-rey requested review from cw118 and removed request for a team March 29, 2024 21:55
@github-actions github-actions bot added the l10n-fr Issues related to French content. label Mar 29, 2024
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Mar 29, 2024 •
edited

Preview URLs (12 pages)
Flaws (33)

Note! 1 document with no flaws that don't need to be listed. 🎉

URL: /fr/docs/Web/API/Element/setAttribute
Title: Element.setAttribute()
Flaw count: 7

  • macros:
    • wrong xref macro used (consider changing which macro you use)
    • wrong xref macro used (consider changing which macro you use)
    • wrong xref macro used (consider changing which macro you use)
  • broken_links:
    • Can't resolve /fr/docs/Tools/Page_Inspector
    • Is currently http:// but can become https://
    • Is currently http:// but can become https://
  • translation_differences:
    • Differences in the important macros (1 in common of 5 possible)

URL: /fr/docs/Web/API/Element/setAttributeNodeNS
Title: element.setAttributeNodeNS
Flaw count: 2

  • broken_links:
    • Is currently http:// but can become https://
  • translation_differences:
    • Differences in the important macros (1 in common of 3 possible)

URL: /fr/docs/Web/API/Element/getAttributeNode
Title: element.getAttributeNode
Flaw count: 7

  • broken_links:
    • Can't resolve /fr/DOM/element.parentNode
    • Can't resolve /fr/DOM/element.previousSibling
    • Can't resolve /fr/DOM/element.nextSibling
    • Can't resolve /fr/DOM/element.getAttribute
    • Is currently http:// but can become https://
    • and 1 more flaws omitted
  • translation_differences:
    • Differences in the important macros (1 in common of 3 possible)

URL: /fr/docs/Web/API/Element/getAttributeNS
Title: element.getAttributeNS
Flaw count: 2

  • macros:
    • /fr/docs/Web/API/Element/hasAttributeNS_ does not exist
  • broken_links:
    • Can't resolve /fr/Add-ons/Code_snippets/getAttributeNS

URL: /fr/docs/Web/API/Element/hasAttributeNS
Title: element.hasAttributeNS
Flaw count: 2

  • broken_links:
    • Is currently http:// but can become https://
  • translation_differences:
    • Differences in the important macros (1 in common of 3 possible)

URL: /fr/docs/Web/API/Element/setAttributeNS
Title: element.setAttributeNS
Flaw count: 2

  • broken_links:
    • Is currently http:// but can become https://
  • translation_differences:
    • Differences in the important macros (1 in common of 3 possible)

URL: /fr/docs/Web/API/Element/removeAttribute
Title: Element.removeAttribute()
Flaw count: 1

  • broken_links:
    • Can't resolve /fr/docs/Web/API/DOMString

URL: /fr/docs/Web/API/Element/removeAttributeNS
Title: element.removeAttributeNS
Flaw count: 2

  • broken_links:
    • Is currently http:// but can become https://
  • translation_differences:
    • Differences in the important macros (1 in common of 3 possible)

URL: /fr/docs/Web/API/Element/setAttributeNode
Title: element.setAttributeNode
Flaw count: 3

  • broken_links:
    • Is currently http:// but can become https://
    • Is currently http:// but can become https://
  • translation_differences:
    • Differences in the important macros (1 in common of 3 possible)

URL: /fr/docs/Web/API/Element/getAttributeNodeNS
Title: element.getAttributeNodeNS
Flaw count: 3

  • broken_links:
    • Is currently http:// but can become https://
    • Is currently http:// but can become https://
  • translation_differences:
    • Differences in the important macros (0 in common of 4 possible)

URL: /fr/docs/Web/API/Element/removeAttributeNode
Title: element.removeAttributeNode
Flaw count: 2

  • broken_links:
    • Is currently http:// but can become https://
  • translation_differences:
    • Differences in the important macros (1 in common of 3 possible)
External URLs (27)

URL: /fr/docs/Web/API/Element/setAttribute
Title: Element.setAttribute()

URL: /fr/docs/Web/API/Element/setAttributeNodeNS
Title: element.setAttributeNodeNS

URL: /fr/docs/Web/API/Element/getAttributeNode
Title: element.getAttributeNode

URL: /fr/docs/Web/API/Element/hasAttributeNS
Title: element.hasAttributeNS

URL: /fr/docs/Web/API/Element/setAttributeNS
Title: element.setAttributeNS

URL: /fr/docs/Web/API/Element/removeAttributeNS
Title: element.removeAttributeNS

URL: /fr/docs/Web/API/Element/setAttributeNode
Title: element.setAttributeNode

URL: /fr/docs/Web/API/Element/getAttributeNodeNS
Title: element.getAttributeNodeNS

URL: /fr/docs/Web/API/Element/removeAttributeNode
Title: element.removeAttributeNode

(comment last updated: 2024-04-14 08:06:39)

SphinxKnight
SphinxKnight requested changes Apr 11, 2024
View reviewed changes
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.

Idéalement, il faudrait limiter la PR au retrait de la macro en question et garder les mises à jour et autres correction pour une/d'autres PR :)
On peut en dscuter :)

@quentin-rey
Copy link
Contributor Author

En effet je n'y ai pas pensé, mais ça paraît plus logique.

Je ne suis pas encore assez à l'aise avec Git, il faut que je reparte du main et que je créé 2 branches ?
Ou j'ai moyen de repartir de ce qui a été fait sur cette branche ?
Je vais essayer de voir

@SphinxKnight
Copy link
Member

SphinxKnight commented Apr 12, 2024 •
edited

Vu que adbebe0 est le commit le plus "proche" de ça mais qu'il contient également d'autres modifications (que le retrait de la macro), je ne sais pas s'il existe une solution super-propre avec Git qui permettrait de tout avoir nickel sur deux branches.

À vue de nez, je dirais :

  1. Créer une nouvelle branche à partir de la branche courante afin de préparer la future PR (celle avec les màj complémentaires) (et la pousser sur le fork histoire d'éviter de perdre le boulot en cas de mauvaise manip en local)
  2. Sur la branche courante (remove_DOMAttributeMethods_macros), revenir à l'état antérieur et faire "à la main" la modification pour retirer les macros. Force-pusher sur le fork remote afin que cette PR soit à jour
  3. Attendre le merge (promis on va faire ça en moins de deux semaines ^ ^)
  4. Rebase la branche créée en 1 sur l'état de la branche main (en étant sur la branche créée en 1, lancer un truc du style git rebase main --strategy-option theirs qui permet de garder la version courante du fichier (vu qu'il y aura déjà les macros de supprimées, ça devrait être bon)
  5. Force-push cette branche sur ton fork et faire une PR correspondante (et là je viendrai à nouveau pinailler ^ ^)

@quentin-rey quentin-rey force-pushed the remove_DOMAttributeMethods_macros branch from b28235c to 9249e4e Compare April 12, 2024 08:43
@quentin-rey
Copy link
Contributor Author

quentin-rey commented Apr 12, 2024 •
edited

J'ai du faire une mauvaise manip, j'ai fais un git push --force-with-lease après avoir remis à la branche au niveau du main, mais avant de faire les suppressions de macros. Y a-t-il moyen de la rouvrir ?

Pour le point 1, j'ai créé la branche :
https://github.com/quentin-rey/translated-content/tree/fr_minor_fixes_and_changes

@SphinxKnight
Copy link
Member

Si tu as toujours les commits en local dans les répertoires git oui, le mieux serait de connaître leur hash / ref pour recréer une branche dessus. En distant, je n'ai pas l'impression de voir grand chose : /

Je n'ai jamais utilisé mais git log --walk-reflogs pourra peut-être te permettre de le retrouver.

@SphinxKnight
Copy link
Member

j'ai retrouvé le hash 44ac224 :)

@quentin-rey
Copy link
Contributor Author

Ce ne serait pas plus simple que je recrée un PR à partir de cette branche que j'ai retravaillé en local ? 🤔
J'ai l'impression que quoi qu'il en soit celui-ci est clos.

@SphinxKnight
Copy link
Member

il y a plusieurs choses :

  • les PR : je peux rouvrir celle-là ou on peut voir ça sur une autre, peu d'importance
  • l'historique/les commits à retrouver si besoin pour faire ce que tu veux (auquel cas le hash te donne accès au fil de commits (permet de remonter les parents si besoin)

@SphinxKnight SphinxKnight reopened this Apr 14, 2024
SphinxKnight
SphinxKnight approved these changes Apr 14, 2024
View reviewed changes
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.

Merci @quentin-rey !

@SphinxKnight SphinxKnight merged commit 6c6a258 into mdn:main Apr 14, 2024
12 checks passed
@quentin-rey quentin-rey mentioned this pull request Apr 14, 2024
Merged
@quentin-rey quentin-rey deleted the remove_DOMAttributeMethods_macros branch April 17, 2024 12:58
ciyoe pushed a commit to ciyoe/translated-content that referenced this pull request Apr 20, 2024
@quentin-rey @ciyoe
fr: remove the {{DOMAttributeMethods}} macros - Fixes mdn#19044 (md…
c534004 
…n#19081)

* remove DOMAttributeMethods macro

* remove DOMAttributeMethods macro
@tristantheb tristantheb added this to ✅ Done in l10n-fr May 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SphinxKnight SphinxKnight SphinxKnight approved these changes

@cw118 cw118 Awaiting requested review from cw118 cw118 is a code owner automatically assigned from mdn/yari-content-fr

Assignees
No one assigned
Labels
l10n-fr Issues related to French content.
Projects
l10n-fr
  
✅ Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

fr: remove the {{DOMAttributeMethods}} macros
2 participants
@quentin-rey @SphinxKnight