Skip to content

Conversation

@utzcoz
Copy link
Contributor

@utzcoz utzcoz commented Dec 14, 2025

Test it on Pixel 9 Fold Emulator, and it occupied full screen.

Before:
Screenshot From 2025-12-14 17-11-16

After:
Screenshot From 2025-12-14 17-11-35

Test it on Pixel 9 Fold Emulator, and it occupied full screen.

Signed-off-by: utzcoz <utzcoz@outlook.com>
@utzcoz
Copy link
Contributor Author

utzcoz commented Dec 16, 2025

Hi @michaelneale, please take a look. Thanks.

android:exported="true"
android:label="Goose Mobile Launcher"
android:launchMode="singleTask"
android:screenOrientation="portrait"
Copy link

@bricefriha bricefriha Jan 11, 2026

Choose a reason for hiding this comment

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

I might be totally in the wrong, be setting android:screenOrientation to its default value won't cause any issue?

Why did we force it to "portrait" in the first place?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

See https://developer.android.com/develop/ui/compose/quick-guides/content/restrict-app-orientation-on-phones#orientation_setting.

screenOrientation is ignored (or is equivalent to unspecified) when apps run in multi‑window mode. So, if you restrict your app to one orientation, the app might not work properly in multi‑window mode.

Normally, it will be considered by system as it is not in multi-window mode.

See https://developer.android.com/guide/topics/manifest/activity-element#screen,
If the app is targeted with SDK 36, the value will be ignored, but the goose-mobile uses targetSdk 34 now.

If the goose-mobile requires portrait for old devices with some reasons, it can be kept, otherwise I recommend removing it.

Choose a reason for hiding this comment

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

thanks @utzcoz, sounds good!

@michaelneale
Copy link
Collaborator

nice - sorry not sure how I missed this.

@michaelneale michaelneale merged commit ad69be2 into block:main Jan 12, 2026
2 checks passed
@utzcoz utzcoz deleted the launcher-for-large-screen branch January 13, 2026 03:16
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