ライティングのリンク切れ修正と、リンク切れへの対策
#1483
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
✅ Deploy Preview for smarthr-design-system ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
修正もれの修正
versionfive
reviewed
Jan 7, 2025
Comment on lines
16
to
20
| 1. [質問（タイトル）](#question-title) | ||
| 2. [リード](#lead-section) | ||
| 3. [原因（見出し）](#cause-heading) | ||
| 4. [説明](#explanation) | ||
| 5. [対処方法](#solution) |
There was a problem hiding this comment.
今回の対応の意図をあまり把握していないの的外れかもしれませんが、ここなどページ内リンクの目的であれば他のプロダクトのコンテンツ同様にh3-nでリンクするのでは問題があるのでしょうか？
There was a problem hiding this comment.
ChatGPTに見出しへのアンカー設定をお願いしたら、ご丁寧にページ内リンクについても設定しなおしてくれた
→ 見出しの順番が変われば機能しなくなるというのはページ内リンクでも同様なので、提案をそのまま通した
という経緯になります！
There was a problem hiding this comment.
デザインパターンなど他のページでその形を取っているので、特になんども修正したり、その見出しを固定のリンクとして何度も参照したいなどのUIテキストのガイドラインのようなニーズがなければ不要な気がしました。
例：https://smarthr.design/products/design-patterns/smarthr-table/
There was a problem hiding this comment.
ありがとうございます、承知です！ ページ内リンクは元に戻しました 🙏
たしかに |
versionfive
approved these changes
Jan 9, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
課題・背景
ライティング内の構成変更にともなうリンク切れが発生しているので直したいhttps://smarthr.atlassian.net/browse/SD-960
やったこと
ライティング内の主要な見出しにアンカー用の{<a id="hoge" />}を追加したやらなかったこと
ライティング内以外の修正やリンクチェック動作確認