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

Update data.mdx #1242

Merged
merged 77 commits into from
Oct 10, 2024
Merged

Update data.mdx #1242

merged 77 commits into from
Oct 10, 2024

Conversation

INABAshina
Copy link
Contributor

@INABAshina INABAshina commented Jul 8, 2024

課題・背景

  • ライティングガイドラインの構成がわかりづらいため、全体的に調整をしました。
  • あわせてAirtableからの生成をしていた用字・用語ページをマークダウン化しています。
    https://smarthr.atlassian.net/browse/SD-779

やったこと

  • 「ライティングガイドライン」の中身を「ライティング」ページに移動
  • 「ライティングスタイル」を「基本方針」に変更
    • 具体の表記ルール的なものは用字用語へ移動
  • 用字用語の内容を以下に変更。UIテキストにあるものも該当のものはこちらに移動。Airtableではなく、マークダウン表記に変更
    • 索引
    • 用字一覧
      • 漢字とひらがなの使いわけ
      • カタカナ語の表記
      • 数字・数えの表記
      • 記号の表記
    • 用語一覧
      • SmartHRの用語
      • コンポーネントの呼び方
      • 操作を表す用語
      • その他類似する用語の使いわけ
    • コンポーネントなどにあるUIテキストに関わるライティングを「ライティング>UIテキスト」のガイドラインとしてまとめる
       - 関係するコンポーネントへのリンク設置
    • ヘルプセンターをヘルプページとリリースノートに分解

やらなかったこと

  • Airtableに関する不要なコードの削除はAstro対応時に削除いただけますと幸いです。

動作確認

UXW内での見た目の確認は完了しているため、アプループだけお願いしたいです!

キャプチャ

Before After

Copy link

netlify bot commented Jul 8, 2024

Deploy Preview for smarthr-design-system ready!

Name Link
🔨 Latest commit 95e7cd9
🔍 Latest deploy log https://app.netlify.com/sites/smarthr-design-system/deploys/67064706776877000815ce99
😎 Deploy Preview https://deploy-preview-1242--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.

@misatokondo
Copy link
Contributor

@INABAshina 組み換え作業ありがとうございます! 「ヘルプページ」配下のコンテンツを確認し、リンクが古いところにいくつか修正コミット入れました🙏

INABAshina and others added 17 commits September 19, 2024 10:14
@INABAshina INABAshina marked this pull request as ready for review October 9, 2024 09:06
@INABAshina INABAshina requested review from versionfive, tosiiu and a team as code owners October 9, 2024 09:06
@INABAshina INABAshina requested review from oti and o-kwr October 9, 2024 09:06
@INABAshina INABAshina merged commit 9110241 into main Oct 10, 2024
5 checks passed
@INABAshina INABAshina deleted the SD-781 branch October 10, 2024 04:22
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.

7 participants