Full screen media no longer exited with a tap

Open
#14,949 9 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
3/5
Estimated time
1-2 days
Newbie friendliness
55/100
Issue type
Bug
Clarity
Mostly clear
Activity status
Active
Tech stack
android, kotlin
Domain
mobile

Research direction

Reproduce the issue on an Android 10 device by opening media in full-screen mode and tapping the screen. Trace the full-screen media tap and navigation-bar handling; done means a tap restores the navigation bar and exits full-screen mode, while the existing swipe behavior remains intact.

Written by the indexing model from the issue text.

Description

Guidelines
  • I have searched open and closed issues for duplicates
  • I am submitting a bug report for existing functionality that does not work as intended
  • This isn't a feature request or a discussion topic
Bug description

Device: 2018 Nokia 6.1 running Android 10

As of somewhere around Signal version 8.21, it was no longer possible to exit full screen media mode (restore the navigation bar with the back button) by tapping anywhere on the screen. I discovered by accident that full screen mode can be exited by swiping up from the bottom of the screen or down from the top of the screen. Having never thought to try this in the past I don't know if these swipes are new. Regardless, tapping is much quicker and less fiddly. The regression is still present in Signal 8.22.2.

I'm happy to provide further information if it will help.

Screenshots

No response

Device

Nokia

Android version

10

Signal version

Around 8.21

Link to debug log

No response

Dominant language
Kotlin
Stars
29.4k
Forks
6.9k
No merged PRs in 30d

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from signalapp/Signal-Android

All issues in signalapp/Signal-Android

Similar issues

More Kotlin issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.