Repository navigation
fix(signage): 起動時に全画面にならない問題を修正 - #218
Merged
Merged
Conversation
Component.onCompleted で表示直後に visibility を切り替えていたため、 環境によっては全画面要求が無視され通常ウィンドウのまま起動していた。 Window の visibility で表示前に全画面を指定する。 Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
yabuta
marked this pull request as draft
October 20, 2026 08:59
yabuta
marked this pull request as ready for review
October 20, 2026 23:58
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
概要
Ubuntu 24.04 環境でサイネージを起動すると、全画面にならず通常ウィンドウのまま表示される問題を修正します。
原因
main.qmlでは、子ItemのComponent.onCompletedでwindow.visibility = "FullScreen"を設定していました。この時点では
Windowのvisible: trueによる表示処理がすでに始まっています。そのため、表示途中のウィンドウに全画面要求を送る形になり、環境 (Qt 5.15.13 / mutter 46) によってはこの要求が無視されていました。表示後に Enter キーで全画面にする操作は従来どおり動作するので、全画面切り替えの処理自体には問題ありません。
変更内容
Windowにvisibility: Window.FullScreenを宣言的に指定し、表示前に全画面状態が決まるようにしたComponent.onCompletedによる全画面設定を削除したvisible: trueは残している。visibilityだけを指定すると、Qt 5.15 でConflicting properties 'visible' and 'visibility'の警告が出るため確認
main.qmlを読み込み、起動直後のvisibilityがFullScreenになり、警告が出ないことを確認🤖 Generated with Claude Code