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

[SD-1005]同期のガイドラインを追加する #1515

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

Conversation

hebikox
Copy link
Contributor

@hebikox hebikox commented Mar 11, 2025

課題・背景

やったこと

やらなかったこと

動作確認

  • Previewでみてね。

キャプチャ

Before After
スクリーンショット 2025-03-11 13 58 54

Copy link

netlify bot commented Mar 11, 2025

Deploy Preview for smarthr-design-system ready!

Name Link
🔨 Latest commit eec9237
🔍 Latest deploy log https://app.netlify.com/sites/smarthr-design-system/deploys/67cfc27fdfe8ef00091a6eb1
😎 Deploy Preview https://deploy-preview-1515--smarthr-design-system.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.

@hebikox hebikox requested a review from a team as a code owner March 11, 2025 04:56
Copy link
Contributor

@drivesketch drivesketch left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@uknmr uknmr left a comment

Choose a reason for hiding this comment

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

純粋な質問です。

  • 元々の課題は何ですか?
  • 画面のコンテキストに依って「再取得」の方がよさそう、ということはないですか?

「こう決まりました!」という結果だけ書くのではなく「なぜなのか」を併記すると学習性が高まると思います。

## 「同期」と表記し、「再取得」の使用は控える
### 説明
データを取り込んで、取得元と同じ状態にすることを「データを同期する」と表記します。
Copy link
Collaborator

Choose a reason for hiding this comment

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

どこで使うことを想定しているのかわかっていないんですが、「〜する」まで必要ですか?

Suggested change
データを取り込んで、取得元と同じ状態にすることを「データを同期する」と表記します。
データを取り込んで、取得元と同じ状態にすることを「データを同期」と表記します。

Copy link
Contributor

Choose a reason for hiding this comment

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

汎用的な用事用語なら↓でもいいのかも

Suggested change
データを取り込んで、取得元と同じ状態にすることを「データを同期する」と表記します。
データを取り込んで、取得元と同じ状態にすることを「同期」と表記します。

@@ -113,6 +113,9 @@ AppNaviは以下の要素で構成されています。

`desktopNavigationAdditionalContent`propsを利用して表示します。

##### 関連ページ
- [「同期」と表記し、「再取得」の使用は控える](/products/contents/idiomatic-usage/action/#use-synk)
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- [「同期」と表記し、「再取得」の使用は控える](/products/contents/idiomatic-usage/action/#use-synk)
- [「同期」と表記し、「再取得」の使用は控える](/products/contents/idiomatic-usage/action/#use-sync)

{/* textlint-enable */}

{<span id="use-synk" />}
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
{<span id="use-synk" />}
{<span id="use-sync" />}

## 「同期」と表記し、「再取得」の使用は控える
### 説明
データを取り込んで、取得元と同じ状態にすることを「データを同期する」と表記します。
Copy link
Contributor

Choose a reason for hiding this comment

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

汎用的な用事用語なら↓でもいいのかも

Suggested change
データを取り込んで、取得元と同じ状態にすることを「データを同期する」と表記します。
データを取り込んで、取得元と同じ状態にすることを「同期」と表記します。

@@ -127,6 +127,7 @@ ignoreH3Nav: true
| 展開 | 解凍, 開く | [ZIPファイルの「展開」と表記し、「解凍」または「開く」の使用は控える](/products/contents/idiomatic-usage/action/#use-extract-for-zip-files) |
| テンプレート | フォーマット | [「テンプレート」と「フォーマット」の使い分け](/products/contents/idiomatic-usage/other/#use-template-and-avoid-format) |
| 問い合わせ | 問合せ | [送りがなをつけて表記する](/products/contents/idiomatic-usage/hiragana/#attach-okurigana) |
| 同期 | 再取得 | [「同期」と表記し、「再取得」の使用は控える](/products/contents/idiomatic-usage/action/#use-synk)|
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
| 同期 | 再取得 | [「同期」と表記し、「再取得」の使用は控える](/products/contents/idiomatic-usage/action/#use-synk)|
| 同期 | 再取得 | [「同期」と表記し、「再取得」の使用は控える](/products/contents/idiomatic-usage/action/#use-sync)|

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.

4 participants