Add: 1~10番目のキャラクターを選択するホットキーを追加 #2034
Conversation
There was a problem hiding this comment.
LGTM！！
ちょっとこの後コメントした内容についてのコミットをさせていただくので、 @tsym77yoshi さんから逆にレビューいただけると･･･！！
（良さそうとか、こここうすると〜みたいなコメントいただければ！）
変更は多分こちらから見れます：
523edeb?w=1
src/type/preload.ts
Outdated
| return { | ||
| action: `${index + 1}番目のキャラクターを選択` as HotkeyActionNameType, | ||
| combination: HotkeyCombination( | ||
| !isMac ? "Ctrl " + roleKey : "Meta " + roleKey, |
There was a problem hiding this comment.
こうできそう！
| !isMac ? "Ctrl " + roleKey : "Meta " + roleKey, | |
| (!isMac ? "Ctrl " : "Meta ") + roleKey; |
src/components/Talk/AudioCell.vue
Outdated
| @@ -490,6 +495,30 @@ const removeCell = async () => { | |||
| } | |||
| }; | |||
|
|
|||
| // N番目のキャラクターを選ぶ | |||
| const selectCharacterAt = (index: number) => { | |||
| if (userOrderedCharacterInfos.value.length >= index + 1) { | |||
src/components/Talk/TalkEditor.vue
Outdated
| registerHotkeyWithCleanup({ | ||
| editor: "talk", | ||
| enableInTextbox: true, | ||
| name: `${i + 1}番目のキャラクターを選択` as HotkeyActionNameType, |
There was a problem hiding this comment.
番目のキャラクターを選択という文字列がasとともに使われていて、元のほうが変わったときに気付けないな〜と思いました。
共通化して番目のキャラクターを選択という文字列をconstで持っておいても良いかも･･･？
|
ありがとうございます！いいと思います！ |
|
非常に丁寧な進行ありがとうございました！！ 結構望まれていた機能だと思うのでツイートしてみました！ 他にも「次・前のキャラクターに変更」するショートカットや、スタイルを選択するショートカットなども便利だと思います！ |
|
ありがとうございます！
|
|
@tsym77yoshi issue作成ありがとうございます！！ 上下キーの仕様は失念していました。。 か、keystrokeライブラリに変えるか、もろもろ頑張って自作するかですかね･･･！ |
内容
キャラクターのデフォルトスタイルを取得する関数を@domain/talkに移動するcommitをmerge
トークで1~10番目のキャラクターを選択するホットキーを追加
関連 Issue
close: #1187
ref: #1896