PageIndexの修正

[wentzzz]

課題・背景

• PageIndexがリンクとして認識しづらい
• indexにソートが効いておらず表示順がデタラメになっている

やったこと

• リンクのスタイルをBoxLinkを参考にしたものに変更してみた
• titleでソートされるように修正

やらなかったこと

動作確認

• https://deploy-preview-1160--smarthr-design-system.netlify.app/products/components/layout/

[netlify]

✅ Deploy Preview for smarthr-design-system ready!

Name	Link
🔨 Latest commit	ca32eb9
🔍 Latest deploy log	https://app.netlify.com/sites/smarthr-design-system/deploys/6646ad0853ff4700099cc299
😎 Deploy Preview	https://deploy-preview-1160--smarthr-design-system.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

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

[Qs-F]

良さそうな気がしました 👍

[ayumizu]

意匠としてはLGTMです！

[uknmr]

ただ下線が引かれたリンクでも良さそうに思ったんですが、BoxLink に近い意匠を増やした方がよさそう、という判断ですか?

[wentzzz]

BoxLink に近い意匠を増やした方がよさそう、という判断ですか?

その認識であっています。理由は、私がこの機能をnext/prevのショートカットと捉えたからですね。
PageIndexをこの機能のショートカットを捉えるならば、意匠を合わせた方が地続きのコンテキストが継承されたショートカットと認識しやすいと思ったからです。

一方で、この意匠によって特定のユースケース（階層の地続きでないページのIndexとして利用する場合など）で使いづらくなるかもなとも思うので、下線＋TEXT_LINKの方が無難なきもしてきました。

[wentzzz]

下線だけに変えてみた

before:

After:

[ayumizu]

LGTM