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: CharacterButtonのメニューが常に下方向に展開されるようにする #1291

Merged
merged 4 commits into from
Apr 27, 2023

Conversation

sabonerune
Copy link
Contributor

内容

キャラクター選択メニューが常に下方向に展開されるようにすることで以下の問題を解決します。

  • 選択しやすいように先頭の方に配置したキャラクターが逆に選択しにくくなる。
  • タイトルバーにメニューが被りタイトルバーに操作が吸われてしまう。

関連 Issue

その他

ウインドウサイズを変更したときは可能な限り追従するようになっています。

@@ -291,6 +294,18 @@ const reassignSubMenuOpen = debounce((idx: number) => {
arr[idx] = true;
subMenuOpenFlags.value = arr;
}, 100);

const buttonRef: Ref<InstanceType<typeof QBtn> | undefined> = ref();
const heightLimit = "65vh"; // QMenuのデフォルト値
Copy link
Contributor Author

Choose a reason for hiding this comment

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

とりあえず元々の動作は変えないようにしました。
参照する方法がなさそうなのでハードコートしました。

src/components/CharacterButton.vue Show resolved Hide resolved
@sabonerune sabonerune marked this pull request as ready for review April 18, 2023 12:57
@sabonerune sabonerune requested a review from a team as a code owner April 18, 2023 12:57
@sabonerune sabonerune requested review from y-chan and removed request for a team April 18, 2023 12:57
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です!
ちょっとコードの可読性的な点をコメントしたので、対応していただけると...!

src/components/CharacterButton.vue Show resolved Hide resolved
@Hiroshiba Hiroshiba requested a review from y-chan April 25, 2023 16:55
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です!

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

@Hiroshiba Hiroshiba merged commit 3a5f7e9 into VOICEVOX:main Apr 27, 2023
@sabonerune sabonerune deleted the force-down-menu branch April 27, 2023 14:34
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