Skip to content

fix: drop manifest screenOrientation portrait (#1171) - #1227

Open
pradhyumna186 wants to merge 1 commit into
Greenstand:masterfrom
pradhyumna186:fix/1171-remove-orientation-resizability-restrictions
Open

fix: drop manifest screenOrientation portrait (#1171)#1227
pradhyumna186 wants to merge 1 commit into
Greenstand:masterfrom
pradhyumna186:fix/1171-remove-orientation-resizability-restrictions

Conversation

@pradhyumna186

Copy link
Copy Markdown
Contributor

Summary

Removes manifest android:screenOrientation="portrait" from TreeTrackerActivity and ImageCaptureActivity so the app no longer declares orientation locks that conflict with Google Play guidance for large screens and Android 16+ behavior. Previous values are preserved as XML comments with context (see AndroidManifest.xml).

Fixes #1171


Checklist

  • Issue first — Tracked in Remove resizability and orientation restrictions in your app #1171 (Google Play / large-screen orientation & resizability).
  • Tests — Not required for this change (manifest-only); no test updates.
  • Lint / analysis — Run locally: ./codeAnalysis (formatting + lint).
  • Docs — Not required for this manifest-only change.

Verification (please review)

Short screen recording attached showing the app rotating between portrait and landscape on a Resizable emulator to see current behavior after removing the manifest lock.

Landscape / large-screen notes (future work, not blocking this PR):

Screen_recording_20260403_102559.webm
Screen_recording_20260403_102535.webm
Screen_recording_20260403_102627.webm

Notes

  • End users are still primarily phones in portrait; this change is to satisfy Play policy and avoid manifest restrictions on tablets / foldables. Follow-up UX polish for non-portrait layouts can be tracked separately.

…nd ImageCaptureActivity to satisfy Google Play large-screen resizability/orientation requirements (Android 16+).
@Elforama

Elforama commented Apr 3, 2026

Copy link
Copy Markdown
Contributor

As mentioned in the related issue, this will not be merged until we have proper support for landscape.

@pradhyumna186

Copy link
Copy Markdown
Contributor Author

I will work on it

@shivanipednekar87-cloud

shivanipednekar87-cloud commented Apr 11, 2026

Copy link
Copy Markdown

Hi, I'm Shivani from the Greenstand design team. I'd like to work on the landscape UI designs for this PR. Happy to be assigned!

@Elforama

Copy link
Copy Markdown
Contributor

Hi @shivanipednekar87-cloud, please talk with Yuri through Slack. She's doing a lot of design work lately.

@pradhyumna186 we'll want designs before work on landscape starts

@cynthiabondi

Copy link
Copy Markdown
Contributor

@shivanipednekar87-cloud , is there any progress here?

@tosinolarinde

Copy link
Copy Markdown

Hi @cynthiabondi

@cynthiabondi

Copy link
Copy Markdown
Contributor

@tosinolarinde please proceed to work on the design for this.

@cynthiabondi

Copy link
Copy Markdown
Contributor

@Elforama

Elforama commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

We should not be designing for the current UI. There will be a UI refresh. We should do it then.

@cynthiabondi cynthiabondi moved this from In Progress to On-Hold in Android Treetracker Jun 16, 2026
@github-actions

Copy link
Copy Markdown

👋 Hi @pradhyumna186, thanks for the contribution!

Before this PR can be reviewed and merged, please add:

  • a 🖼️ screenshot of the UI change (or mark the PR as non-ui if there is no UI change)

Just drag-and-drop the file(s) into the PR description and GitHub will upload them. This check re-runs automatically when you edit the description.

If this change has no user-visible effect, tick the "no user-visible / UI effect" box in the description (or ask a maintainer to add the non-ui label) to skip the screenshot requirement. A video is still required even with no UI changes, to show that the affected parts of the app still work.

@pradhyumna186

Copy link
Copy Markdown
Contributor Author

I have the landscape design. I will start working on it @cynthiabondi

@cynthiabondi

Copy link
Copy Markdown
Contributor

@pradhyumna186 please discuss with @Elforama first as he mentioned there will be a UI refresh

@pradhyumna186

Copy link
Copy Markdown
Contributor Author

okay @cynthiabondi I will wait on @Elforama for confimation

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: On-Hold

Development

Successfully merging this pull request may close these issues.

Remove resizability and orientation restrictions in your app

5 participants