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

fix: 「ref で DOM を操作する」ページで画像が表示できていない事象を解消 #798

Closed

Conversation

nekonado
Copy link
Contributor

@nekonado nekonado commented Oct 6, 2024

作業内容

  • 画像取得先の https://placekitten.com が521エラーで画像をレスポンスできない状態になっており、対象ページで画像が表示されない(結果としてrefフックの動作確認ができない)事象が発生していたためこれに対応。
  • オリジナルページ では https://placecats.com から画像取得するようになっていたため、日本語版ページでも同様に修正。

対象ページ

https://ja.react.dev/learn/manipulating-the-dom-with-refs

📸 作業後の状況

react-ref.mov

対象ページ: https://ja.react.dev/learn/manipulating-the-dom-with-refs
画像取得先の placekitten.com が521エラーで画像をレスポンスしない状態になっており、画像が表示されない事象が発生していたためこれに対応。
オリジナル(英語版)ページでは placecats.com から画像取得するようになっていたため、日本語版ページでも同様に修正。
Copy link

vercel bot commented Oct 6, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
legacy-ja-reactjs-org ⬜️ Ignored (Inspect) Visit Preview Oct 6, 2024 6:37am

@smikitky
Copy link
Member

smikitky commented Oct 7, 2024

ありがとうございます。

#799 のような自動生成スクリプトで定期的に本家ページと同期を取るようになっています。仕組みはこちらをご覧ください。

placecats.com への修正は reactjs/react.dev#7189 で 10 月 1 日に本家で修正済みですので、#799 で本家と同期をとるタイミングで同時に修正予定です。よほど緊急性がない限り、それ以外でバラバラに個別に作業をすると管理不可能となりますので、申し訳ありませんがこちらはクローズさせていただきます。もう少しお待ちください。

@smikitky smikitky closed this Oct 7, 2024
@nekonado nekonado deleted the fix/images-not-displaying-on-ref-page branch October 7, 2024 08:43
@nekonado
Copy link
Contributor Author

nekonado commented Oct 7, 2024

@smikitky

ご確認ありがとうございます!

更新の仕組みについても添付のZennの記事を読んで理解することができました。
いつもReactのドキュメント翻訳に貢献いただきありがとうございます。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants