Skip to content

Comments

fix: call controls visibility after closing PiP mode [#WPB-15142]#3770

Merged
sergeibakhtiarov merged 1 commit intodevelopfrom
fix/call-controls-in-pip-mode
Dec 27, 2024
Merged

fix: call controls visibility after closing PiP mode [#WPB-15142]#3770
sergeibakhtiarov merged 1 commit intodevelopfrom
fix/call-controls-in-pip-mode

Conversation

@sergeibakhtiarov
Copy link
Contributor

@sergeibakhtiarov sergeibakhtiarov commented Dec 27, 2024

BugWPB-15142 [Android] Call controls not visible after restoring the call UI from background

https://wearezeta.atlassian.net/browse/WPB-15142

What's new in this PR?

Issues

Call controls are not visible when returning to the application after closing picture-in-picture mode.

Causes (Optional)

The picture-in-picture activity mode stored in the activity is read directly from the compose code. So the changes in the
picture-in-picture mode are not causing recomposition and app keeps hiding call controls after closing the picture-in-picture mode.

Solutions

Observe picture-in-picture activity mode from compose and store value in compose state.

@sonarqubecloud
Copy link

@github-actions
Copy link
Contributor

Built wire-android-staging-compat-pr-3770.apk is available for download

@github-actions
Copy link
Contributor

Built wire-android-dev-debug-pr-3770.apk is available for download

@sergeibakhtiarov sergeibakhtiarov added this pull request to the merge queue Dec 27, 2024
Merged via the queue into develop with commit f1c158b Dec 27, 2024
11 checks passed
@sergeibakhtiarov sergeibakhtiarov deleted the fix/call-controls-in-pip-mode branch December 27, 2024 11:02
@echoes-hq echoes-hq bot added the echoes: product-roadmap/bug Work contributing to resolve a bug not critical enough to have raised an incident. label Jan 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

echoes: product-roadmap/bug Work contributing to resolve a bug not critical enough to have raised an incident. size/XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants