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-986「スクショに関するガイドラインの修正」 #1494

Merged
merged 2 commits into from
Jan 24, 2025

Conversation

emilien323
Copy link
Contributor

@emilien323 emilien323 commented Jan 23, 2025

課題・背景

  • SD-986
  • ガイドライン定例のNA
     - スクリーンショットのガイドラインの見出しを修正する(スクリーンショットを使ってはいけないわけではないことが伝わるようにする)
     - 手順1にスクリーンショットを使うことを追記する

やったこと

  • 見出しの修正
  • テキストの修正・追記

やらなかったこと

  • チケットのスコープ外の修正

動作確認

  • Previewでみてね。

キャプチャ

Before After

Copy link

netlify bot commented Jan 23, 2025

Deploy Preview for smarthr-design-system ready!

Name Link
🔨 Latest commit 5a587a0
🔍 Latest deploy log https://app.netlify.com/sites/smarthr-design-system/deploys/6792e8d7c0cf55000831ae1a
😎 Deploy Preview https://deploy-preview-1494--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.

Copy link
Contributor

@otapo otapo left a comment

Choose a reason for hiding this comment

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

基本めっちゃ良いと思いました!

1点だけコメントしました。


#### スクリーンショットを使うシーン

操作対象の特定が難しい場合や、操作対象の明示以外に画像で伝えたいことがある場合は、スクリーンショットを使います。

- 一番最初の手順など、操作の開始位置を示す場合
Copy link
Contributor

Choose a reason for hiding this comment

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

ヘルペPDCAの中で定量的なファクトも取れてると思うので、その旨書いちゃうと
より良いかもと思いました。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

「操作の開始位置をテキストだけで説明すると、「操作箇所がわからない」というユーザーの迷いに繋がります。」or「操作の開始位置をスクリーンショットで示すことで、ユーザーが迷わずに操作を開始できます。」みたいな感じですか?

Copy link
Contributor

Choose a reason for hiding this comment

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

です!

より具体的に「問い合わせが増えます」など書いても良いと思います。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

問い合わせが増えるかはわからないので、「操作の開始位置をテキストだけで説明すると、「操作箇所がわからない」というユーザーの迷いにつながります。」を追記しました。もう一つのほうがポジティブかなと思いましたが、迷わないかどうかはわからないので.....

Copy link
Contributor

@hebikox hebikox left a comment

Choose a reason for hiding this comment

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

Approveです!(otapoさんのコメントが反映できそうであればお願いします🙏)

Copy link
Contributor

@otapo otapo left a comment

Choose a reason for hiding this comment

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

LGTM!

@emilien323 emilien323 merged commit 6ebe9cf into main Jan 24, 2025
5 checks passed
@emilien323 emilien323 deleted the SD-986 branch January 24, 2025 01:54
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.

3 participants