Skip to content

[FEATURE REQUEST] Allow passcode and pattern screens in landscape mode#4871

Open
joragua wants to merge 5 commits into
masterfrom
feature/passcode_and_pattern_in_landscape
Open

[FEATURE REQUEST] Allow passcode and pattern screens in landscape mode#4871
joragua wants to merge 5 commits into
masterfrom
feature/passcode_and_pattern_in_landscape

Conversation

@joragua
Copy link
Copy Markdown
Contributor

@joragua joragua commented May 21, 2026

Related Issues

App: -

Starting from Android 16, Android will ignore resizability and orientation restrictions for large screen devices, such as foldables and tablets. That means that the android: screenOrientation configuration in the manifest will be ignored, and screens using that configuration should support both orientations (portrait and landscape). In our case, we have two screens affected by this restriction: passcode and pattern screens

Work done:

  • Passcode

    • New layout for landscape (phones)
    • New layout for landscape (tablets)
    • Renamed w600dp with sw600dp in order to use the layout-land for phones in landscape
  • Pattern

    • New layout for landscape (phones)
    • Renamed w600dp with sw600dp in order to use the layout-land for phones in landscape
Captura de pantalla 2026-05-21 a las 13 59 02
  • Add changelog files for the fixed issues in folder changelog/unreleased. More info here
  • Add feature to Release Notes in ReleaseNotesViewModel.kt creating a new ReleaseNote() with String resources (if required)

QA

Test plan:

Reports:

@joragua joragua self-assigned this May 21, 2026
@joragua joragua added this to the 4.8.1 - Current milestone May 21, 2026
@joragua joragua force-pushed the feature/passcode_and_pattern_in_landscape branch 3 times, most recently from 1d70daa to 6c990bc Compare May 21, 2026 13:02
@joragua joragua force-pushed the feature/passcode_and_pattern_in_landscape branch from 6c990bc to 0cb53a7 Compare May 22, 2026 08:14
@joragua joragua marked this pull request as ready for review May 22, 2026 09:04
@joragua joragua requested a review from jesmrec May 22, 2026 09:04
Copy link
Copy Markdown
Contributor

@jesmrec jesmrec left a comment

Choose a reason for hiding this comment

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

Some comments there, @joragua

Comment thread owncloudApp/src/main/res/layout-land/passcode_lock_activity.xml
Comment thread owncloudApp/src/main/res/layout-land/activity_pattern_lock.xml Outdated
Comment thread owncloudApp/src/main/res/layout-land/passcodelock.xml Outdated
Comment thread owncloudApp/src/main/res/layout-sw720dp-land/passcodelock.xml Outdated
Comment thread owncloudApp/src/main/res/layout-land/passcodelock.xml Outdated
Comment thread owncloudApp/src/main/res/layout-sw720dp-land/passcodelock.xml Outdated
Comment thread owncloudApp/src/main/res/layout-land/passcodelock.xml Outdated
Comment thread owncloudApp/src/main/res/layout-sw720dp-land/passcodelock.xml Outdated
Comment thread owncloudApp/src/main/res/layout-sw600dp/pattern_lock_activity.xml
Comment thread owncloudApp/src/main/res/layout-sw600dp/pattern_lock_activity.xml
@joragua joragua force-pushed the feature/passcode_and_pattern_in_landscape branch from 458e066 to 1464890 Compare May 25, 2026 09:50
@joragua joragua requested a review from jesmrec May 25, 2026 09:58
Copy link
Copy Markdown
Contributor

@jesmrec jesmrec left a comment

Choose a reason for hiding this comment

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

CR approved!! moving to QA

@jesmrec
Copy link
Copy Markdown
Contributor

jesmrec commented May 25, 2026

(1)

  1. Enable passcode
  2. Type 2 characters
  3. Change orientation

Current:

Typed characters are correctly stored, but point indicators are missing on UI

2 characters typed in portrait 3rd character, in landscape
Screenshot_20260525_135119 Screenshot_20260525_135140

Expected:

UI status keeps after orientation changes

Xiaomi Redmi Note 2, Android 15
1464890f

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants