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

ボイボ寮のキャラ個別ページからダウンロードを可能にする #149

Conversation

Hiroshiba
Copy link
Member

@Hiroshiba Hiroshiba commented Mar 12, 2023

の解決です。

image

  • ページ上部にヘッダーを表示
  • 下部にもダウンロードページへの遷移ボタンを表示
  • モーダル形式をやめ通常のページ扱いにし、キャラクター一覧に戻る(進む)ボタンを追加
    • ヘッダーを表示したらモーダルっぽくなくなったため

close #113

本番ページヘは未デプロイです。
プレビューページへはデプロイ済みです。 https://preview--voicevox.netlify.app/dormitory/

@@ -220,7 +220,7 @@ export const useDetailedCharacterInfo = () => {
}

const characterInfos: {
[key in CharacterKey]: CharacterInfo | undefined
[key in CharacterKey]: CharacterInfo
Copy link
Member Author

Choose a reason for hiding this comment

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

運用上undefinedなキャラクターをここに表示する機会がなくなったので消去

@Hiroshiba
Copy link
Member Author

(以前このあたりを実装してくださった @k-chop さんに共有です 🙇‍♂️)

URLのハッシュを使うことで、こちらで議論していたののうち、個別ページ→一覧ページへボタンで進むときはスクロール位置がそこそこ良い感じになります。

ただ↑のコメントにもある通り、gatsbyにはハッシュありURLにブラウザバックしたときにそこへスクロールするバグがあるので、思った挙動になりません。
まあそもそもgastbyのnavigateとスクロール位置は結構変なとこに飛ぶことも多いので、実装的にわかりやすいこちらを優先しようかなと思いました・・・。
(そのキャラクターにスクロールするURLがほしいと思う人もいそうなので、/dormitory/でid付けるのはまあ良いかなと)

@Hiroshiba
Copy link
Member Author

このページ、他のキャラクターの一覧がほしくなりますね・・・。製品ページもそうだけど。
とりあえずメモがてらissue化してみました。

@Hiroshiba
Copy link
Member Author

マージします!!

@Hiroshiba Hiroshiba merged commit 583332c into master Mar 14, 2023
@Hiroshiba Hiroshiba deleted the ボイボ寮のキャラ個別ページからダウンロードを可能にする branch March 14, 2023 19:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant