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

ENH: 表示範囲外のAudioCellがアクティブになったとき表示範囲内までスクロールするようにする #1304

Merged
merged 2 commits into from
May 9, 2023

Conversation

sabonerune
Copy link
Contributor

内容

アクティブなセルが切り替わったときそのセルが表示範囲外なら表示範囲内に自動的にスクロールするように修正します。
これにより連続再生時に再生しているテキスト欄が追従します。

関連 Issue

その他

このツイートを見てボタンのある行を表示範囲外にするつもりでしたが移動範囲が大きくて使いにくくなった気がしたので止めました。

@sabonerune sabonerune requested a review from a team as a code owner May 3, 2023 03:28
@sabonerune sabonerune requested review from y-chan and removed request for a team May 3, 2023 03:28
Copy link
Member

@y-chan y-chan left a comment

Choose a reason for hiding this comment

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

LGTMです!

@Hiroshiba
Copy link
Member

いろいろ試してみました、とても良いですね!!
連続再生に関しては仕様変更のように感じると思うので、ちょっとだけ案内してあげたほうが良いかもと思いました。
アプデが入るよ~ツイートしてみて反応を見てみようと思います。

以下試したことと判断メモ

  • テキスト欄追加
    • OK。
  • undo/redo
    • activeは変わらないので移動しない。OK。
  • 連続再生
    • スクロールするという仕様変更になる。それでOKだと思う。
    • 連続再生が終わったとき元の位置に戻る。それでOKだと思う。
  • ↑↓キーで移動したとき
    • OK。
  • deleteしたとき
    • OK。
  • escapeキー・enterキーを押したとき
    • activeは変わらないので移動しない。OK。
  • 複数行テキストをコピーしたとき
    • 一番下に移動する。OK。

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!!

src/views/EditorHome.vue Outdated Show resolved Hide resolved
@Hiroshiba Hiroshiba merged commit d4cd95a into VOICEVOX:main May 9, 2023
@sabonerune sabonerune deleted the enh/audiocell-scroll branch May 9, 2023 10:53
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.

連続再生している時、表示エリアを再生しているテキスト欄に画面を追従させる
3 participants