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

patterns.html の簡易版 #154

Open
wants to merge 3 commits into
base: waic-main
Choose a base branch
from
Open

patterns.html の簡易版 #154

wants to merge 3 commits into from

Conversation

ef81sp
Copy link

@ef81sp ef81sp commented Nov 30, 2024

関連Issue

なし
2024/11/28の定例会で会話したものです

レビュワーに見てもらいたい箇所

  • sharedディレクトリ配下のうち、patternsから参照しているファイルも最新化しています。
  • 不要な箇所について、コメントアウトしておいた方がのちの翻訳で便利かもしれないと思いつつ、ユーザーに(開発者ツールで)見えてしまうし、gitで履歴を遡れるので、どかんと消してしまいました。コメントアウトの方がよさそうでしたら、そのように直そうと思います。

翻訳チェックリスト

  • html要素にlang="ja"属性を付与する
  • h1要素の直下に翻訳元リビジョンを記載する

@ef81sp ef81sp requested a review from bakera November 30, 2024 11:12
Copy link

netlify bot commented Nov 30, 2024

Deploy Preview for waic-apg ready!

Name Link
🔨 Latest commit ee2d1c6
🔍 Latest deploy log https://app.netlify.com/sites/waic-apg/deploys/674af328ecdf7b000803de15
😎 Deploy Preview https://deploy-preview-154--waic-apg.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

<div class="tile-introduction">
A window splitter is a moveable separator between two sections, or panes, of a window that enables users to change the relative size of the panes.
<div class="pattern-introduction">
アラートは、利用者のタスクを中断することなく、利用者の注意を引くように、簡潔で重要なメッセージを表示する要素です。
Copy link
Author

Choose a reason for hiding this comment

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

alert-pattern.html の1文目をコピペしてきました

Copy link
Author

@ef81sp ef81sp Nov 30, 2024

Choose a reason for hiding this comment

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

PRごとに最新版に追従する手順 の影響で、差分がずれますね。。
(変更前は古いファイルの翻訳前、変更後は新しいファイルの翻訳後)

waic-mainが分岐してから1年以上経っているので(分岐元: e716cc2)、一度最新版にrebaseした方がよいかもしれません。

@bakera
Copy link
Contributor

bakera commented Dec 20, 2024

@bakera
Copy link
Contributor

bakera commented Dec 20, 2024

コメントアウトはせずに容赦無く削除で良いと思います。
多分こんな感じで問題ないんじゃないですかね。

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

Successfully merging this pull request may close these issues.

2 participants