フルスクリーンモードを追加(#2251) #2273
フルスクリーンモードを追加(#2251) #2273
Conversation
|
@Hiroshiba |
| subMenu: [...props.viewSubMenuData], | ||
| subMenu: [ | ||
| ...props.viewSubMenuData, | ||
| { type: "separator" }, |
There was a problem hiding this comment.
（細かいですが）トークエディタだと一番上にseparatorが来そう？
There was a problem hiding this comment.
ほんとですね！！ ちょっと見た目変になりそう。
残しといていただいたら後でちょっとこちらで調整させていただきます！ 🙏
There was a problem hiding this comment.
コメントありがとうございます！
@Hiroshiba
個人的な感想として、フルスクリーンモード切替ボタンと他のボタンをセパレーターで区切る意味はそんなにないかなと思います。このPRにおいては、セパレーターを消す方向で修正進めてもいいでしょうか？（一旦残して後から修正入れていただく方向でも大丈夫です！）
There was a problem hiding this comment.
たしかに、一旦なしがとりあえずで一番良さそう！ご提案ありがとうございます！
| @@ -754,7 +754,13 @@ registerIpcMainHandle<IpcMainHandle>({ | |||
| win.maximize(); | |||
| } | |||
| }, | |||
|
|
|||
| TOGGLE_FULLSCREENMODE: () => { | |||
There was a problem hiding this comment.
| TOGGLE_FULLSCREENMODE: () => { | |
| TOGGLE_FULLSCREEN: () => { |
の方がいいかも？（これを変えるなら他のところも変える必要があるはず）
There was a problem hiding this comment.
たしかにFULLSCREENMODE一単語はほんのちょっぴり違和感あるかもですね！
TOGGLE_FULLSCREEN_MODEという手もありそう。
|
@sevenc-nanashi
なるほどですね！！ どうするのが正解かわからないですね･･･。 ちなみにフルスクリーン状態かどうかを取得するには、 |
There was a problem hiding this comment.
ほぼLGTMです！！ プルリクエストありがとうございます！！
せっかくなのでもしよかったら @sevenc-nanashi さんに指摘いただいた箇所の変更お願いできれば･･･！
分からなかったり時間なかったら遠慮なく言ってください！
こちらでちょいとこちらでやらさせていただこうと思います！
| @@ -466,6 +471,7 @@ export const hotkeyActionNameSchema = z.enum([ | |||
| "元に戻す", | |||
| "やり直す", | |||
| "新規プロジェクト", | |||
| "全画面表示/ウィンドウ表示切り替え", | |||
There was a problem hiding this comment.
丁寧な表記いいですね！
ちょっと長いのでこうしちゃうのどうでしょ！
| "全画面表示/ウィンドウ表示切り替え", | |
| "全画面表示の切り替え", |
（slackが「フルスクリーン表示の切り替え」だったので真似してみました）
@Hiroshiba @sevenc-nanashi 個人的にはフルスクリーンモードになったらメニュバー自体が非表示になるのがしっくりくるのですが、正直どちらがいいのかは私も分からないですね…… 実装的なお話をさせて頂くと、@sevenc-nanashi さんの案であればすぐに実装できるかと思います。逆に最大化・最小化ボタンを非表示にしたり、メニューバーを非表示にする案については実装の目途が立っていない状況です（試しにsetMenuBarVisibilityやsetMinimizable、setMaximizableを使って非表示にしようと試みたのですがうまく行かず……）。 @sevenc-nanashi さんの案で進めてしまいたいと思っているのですが、どうでしょうか？ |
メニューバーは出しちゃっていいと思います。 ブラウザとかだと、フルスクリーンは「ページの内容に集中」みたいな意味合いでツールバーが消えてる（ツールバーはページの内容に影響しない）と思うのですが、ボイボだと「画面を大きくする」以外にないと思うんですよね（自分が思いつく限りだと）。 ちなみに、メニューバーを隠すのはMenuBarコンポーネントにv-ifをつけて切り替えればできると思います。（もしかしたら多少レイアウトが崩れるかもしれませんが…） |
|
一般的なガイドラインに合わせるのが良さそうなので、OS側のガイドラインを調べてみました。 Windows側は」フルスクリーンに関するガイドラインがない気がしますね･･･。 ちなみにVSCodeも複雑なメニューバーを持っているのですが、フルスクリーンにするとこんな感じになりました。 なので一般的な方針はなさそうなのでVOICEVOXとしてどうしたいかで決めると良さそう感！ 僕は2人ともの意見どちらにも賛成で、大事だからアクセスできるべきだし、フルスクリーンの記事に従ってメニューを隠すのもありだと思います。 2番目は、大事な機能なのでアクセスは可能にしておく形（つまりメニューバー全てを非表示にはしない形）かな～と思ってます。 このプルリクエストの方針としては、一旦今のメニューバー全体表示される形で着地させるというのはどうでしょうか？ あと最大化ボタンとかをv-ifで隠すのは実装こんな感じです。もしご興味あれば！（放置でもOKです！） |
|
@terapotan お疲れ様です！ |
|
@Hiroshiba 実装方針は固まっていて、実装しようと思えばすぐに出来る状況です。 |
|
いえいえ！！ もしよければいつでも！！ では一応、このプルリクエストは、もし他の方が引き継ぎたかったら自由について OK という感じにさせてください！ 🙏 |
内容
フルスクリーンモードを追加。F11キーもしくは、メニュータブの<表示>-><全画面表示/ウィンドウ表示切替>から、全画面表示とウィンドウ表示を切り替えられるように修正を行った。
関連 Issue
ref #2251
スクリーンショット・動画など
2024-09-27.142127.mp4