Skip to content

Conversation

nebocco
Copy link
Contributor

@nebocco nebocco commented Mar 26, 2022

内容

アクセント句分割の当たり判定を広げてみました。いくつか方法はあると思うのですが、もっとも単純に「文字部分の範囲を削り、分割部分の判定を増やす」という方法を試してみました。全体の表示幅に影響が出ないよう調整しました。

関連 Issue

ref #453

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

image

image

その他

提案の確認用なので merge せずに閉じても構いません。

@Hiroshiba
Copy link
Member

おー!!プルリク作成ありがとうございます!!
時間作ってちょっと試してみます…!

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

お待たせしてすみませんでした。
使いやすそう&コードも問題なさそうなので、勝手ながらDraftを外してマージしたいと思います。

Dreftにしていただいていたのもとても嬉しい配慮でした!
またよければプルリクエストお待ちしています・・・!

@Hiroshiba Hiroshiba marked this pull request as ready for review March 28, 2022 15:09
@Hiroshiba Hiroshiba merged commit 5a301ad into VOICEVOX:main Mar 28, 2022
y-chan added a commit to y-chan/voicevox that referenced this pull request Jun 25, 2022
Hiroshiba pushed a commit that referenced this pull request Jun 26, 2022
* fix design and remove bad css

* apply changes of #772 and #792

* remove unused import
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