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

Add: ボイボ寮キャラクターの呼称のコピー機能 + リファクタリング #168

Merged
merged 14 commits into from
Oct 4, 2023

Conversation

wappon28dev
Copy link
Contributor

@wappon28dev wappon28dev commented Sep 28, 2023

内容

ボイボ寮キャラクターの呼称表 (呼び方一覧表) のコピー機能を実装します.
また, #163 に関連して以下のような修正を行います:

- URL アンカー (#{characterId}) で飛んだときに画面の中心にスクロールするように
- チョットハイライトアニメーション付けました (discord/slack みたいな)

  • ボイボ寮にある呼称表へ飛ばすリンクが call-names になったまま (URLのcall-namesをcall_namesに #167 のミス?) なので call_names
  • セルにホバーしたら, “クリックしてコピー: 「ずんだもん」” のように出てくるようにしてみました

関連 Issue

refs:

スクリーンショット・動画など

準備中

その他

不要な機能があれば遠慮なくお教えください!!!

Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

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

プルリクエストありがとうございます!!
ざっと動かして見させていただきました、コピーボタンとかすごくいい感じに思います!!!
リファクタリングもありがとうございます!

ちょっとまだコードは見れてないのですが、guiを見てのコメントを先に・・・ 🙇

コピーボタンすごくいいと思います!!!
消えるまでのアニメーションがちょっと長めに感じたので短くさせていただくことになるかもです!

横スクロールした時に表示がバグっていそうでした!z-index?
image

アンカーが指定されて飛んできた場合のアニメーションは、ボイボーWebページ全体的にかなり珍しそうなのでおそらくもう少し落ち着いた感じにさせていただくことになっちゃうかもです 🙇
点滅せずに枠がずっとつき続けるようなUIや、あるいは少し長めの時間をかけて薄くなって消えていく色枠とか・・・?
(ちょっとアニメーションデザインに詳しくなく。。。)

あとアンカーがあるけれどもそのリンクを発行するための道がなさそうに感じました!
自分でHTMLタグを見れる人であれば発見できますが、現状だと結構もったいないな~と思います。

一番左のキャラクターをマウスホバーした時、コピーボタンみたいな感じでリンクボタンアイコンが出てくるとかはどうでしょう?
アンカーから飛んできた人が見る枠がプレビューで表示されていると、何へのリンクなのかが分かりやすいかも・・・?
ただまぁ実装が大掛かりになってしまうので、やるやらない含めてお任せしたいなと・・・!!

@wappon28dev
Copy link
Contributor Author

wappon28dev commented Sep 29, 2023

レビューありがとうございます!!!

消えるまでのアニメーションがちょっと長めに感じたので短くさせていただくことになるかもです!

消えるまでの時間を 3s → 1.5s にしてみました.

横スクロールした時に表示がバグっていそうでした!z-index?

ご指摘ありがとうございます!
これについては, アンカーが指定されてきたときに枠 (border / outline) と背景 (background) のスタイルが知らない間に衝突してしまってました.

ところで, アンカーが指定されてきたときに枠をハイライトする機能ですが,

  • ページが表示されている場合でアンカーを踏むと, ブラウザがどうしてもアンカーへ移動してしまう
    → コールバックを貼るのが難しい……
  • 枠のスタイルとセル背景のスタイルが重複して, その回避が私では厳しい
  • この PR での範疇外, そもそも issue に上がっていない

のような理由より, アンカーが指定されているときに枠をハイライトする機能のコミットを Revert し, この PR から機能を削除しました. また,

  • 他ページと馴染むよう, 控えめなアニメーション
  • アンカーリンクを発行する機能
  • その際のプレビューでも枠が見えていると良いかも?

などのコメントありがとうございました! これを踏まえていつか実装するかもしれません.


ということで改めて以下が変更点です. 

  • ボイボ寮にある呼称表へ飛ばすリンクが call-names になったまま (URLのcall-namesをcall_namesに #167 のミス?) なので call_names に
  • セルにホバーしたら, “クリックしてコピー: 「ずんだもん」” のように出てくるように
  • コピー完了のチェックマークはちょっと短めに

よろしくお願いします!!!

Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

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

挙動はlgtmです!!

コード読ませてもらいました、すごいいい感じだと思います!!
ちょっと細かいかもですが2点ほどコメントしてみました!

src/pages/dormitory/call_names.tsx Outdated Show resolved Hide resolved
src/pages/dormitory/call_names.tsx Outdated Show resolved Hide resolved
src/pages/dormitory/call_names.tsx Outdated Show resolved Hide resolved
src/pages/dormitory/call_names.tsx Outdated Show resolved Hide resolved
Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

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

LGTM!!!!!
5日にリリース予定です!その前に1回プレビュー版を更新したいと思います。

ちょっとこちらで微修正だけさせていただきます!

src/pages/dormitory/call_names.tsx Show resolved Hide resolved
@Hiroshiba Hiroshiba merged commit f3025fe into VOICEVOX:master Oct 4, 2023
1 check passed
@wappon28dev wappon28dev deleted the feature/call-names-copy branch October 4, 2023 23:58
@Hiroshiba
Copy link
Member

呼称ページ、リリースしました!! 🎉 🎉 🎉
https://twitter.com/voicevox_pj/status/1709908431224889622

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.

2 participants