Skip to content

Feature: Add minimized video service (YouTube-style PiP) #5423

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 24 commits into
base: feature/bma/minimizeVideoPlayer
Choose a base branch
from
+722 −10
Open

Feature: Add minimized video service (YouTube-style PiP) #5423

Show file tree
Hide file tree
Changes from 15 commits
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
9f024c7
feat(floating-video): add system-wide mini-player draft
aliIsazadeh Sep 26, 2025
a111b5e
Merge branch 'element-hq:develop' into minimized-video
aliIsazadeh Sep 27, 2025
16db3ab
fixed the bug of video being full size at first
aliIsazadeh Sep 27, 2025
bbc5458
Merge remote-tracking branch 'origin/minimized-video' into minimized-…
aliIsazadeh Sep 27, 2025
0f3fec9
Merge branch 'develop' into minimized-video
aliIsazadeh Sep 30, 2025
fe18ef7
-remove unused setMinimize
aliIsazadeh Sep 30, 2025
49a765d
-removed added icons and string and using existing icons and strings
aliIsazadeh Sep 30, 2025
daf7461
-adjusting maximize button size
aliIsazadeh Sep 30, 2025
46d226b
Merge branch 'element-hq:develop' into minimized-video
aliIsazadeh Sep 30, 2025
d07dc1e
-remove extra service declaration of service in app manifest
aliIsazadeh Sep 30, 2025
b99e67f
Merge remote-tracking branch 'origin/minimized-video' into minimized-…
aliIsazadeh Sep 30, 2025
2f9f56d
-remove extra import
aliIsazadeh Sep 30, 2025
7c13c1f
-remove extra string file
aliIsazadeh Oct 2, 2025
b9f415f
refactor: Convert floating video overlay to Compose with direct view …
aliIsazadeh Oct 4, 2025
a90c19a
Merge branch 'develop' into minimized-video
aliIsazadeh Oct 6, 2025
6a3cb61
-revert changes in gradle.properties and project
aliIsazadeh Oct 11, 2025
f91d167
- Broke down large FloatingVideoService into multiple smaller files f…
aliIsazadeh Oct 12, 2025
1466e74
Merge remote-tracking branch 'origin/minimized-video' into minimized-…
aliIsazadeh Oct 12, 2025
b33720c
fix(video): stabilize floating video behavior
aliIsazadeh Oct 13, 2025
d4228a8
-Converted VideoDataRepository to @SingleIn(AppScope::class) with @In…
aliIsazadeh Oct 16, 2025
19defb1
-on video completes the minimized video gose away
aliIsazadeh Oct 17, 2025
2cf92d5
-Toast for display over apps persmission(string needs to be added to …
aliIsazadeh Oct 17, 2025
44105d2
Merge branch 'element-hq:develop' into minimized-video
aliIsazadeh Oct 17, 2025
1978751
Merge branch 'develop' into minimized-video
aliIsazadeh Oct 20, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
35 changes: 35 additions & 0 deletions .idea/codeStyles/Project.xml
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 0 additions & 1 deletion app/src/main/AndroidManifest.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@

<!-- To be able to install APK from the application -->
<uses-permission android:name="android.permission.REQUEST_INSTALL_PACKAGES" />

<application
android:name=".ElementXApplication"
android:allowBackup="false"
Expand Down
6 changes: 5 additions & 1 deletion gradle.properties
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
# http://www.gradle.org/docs/current/userguide/build_environment.html
# Specifies the JVM arguments used for the daemon process.
# The setting is particularly useful for tweaking memory settings.
org.gradle.jvmargs=-Xmx4g -Dfile.encoding=UTF-8 -XX:+UseG1GC
#org.gradle.jvmargs=-Xmx4g -Dfile.encoding=UTF-8 -XX:+UseG1GC

# AndroidX package structure to make it clearer which packages are bundled with the
# Android operating system, and which are packaged with your app"s APK
Expand Down Expand Up @@ -54,3 +54,7 @@ ksp.allow.all.target.configuration=false

# Used to prevent detekt from reusing invalid cached rules
detekt.use.worker.api=true
org.gradle.java.home=/usr/lib/jvm/java-17-openjdk-amd64
org.gradle.jvmargs=-Xmx4g -Dfile.encoding=UTF-8
kotlin.compiler.execution.strategy=in-process
kotlin.daemon.jvm.options=-Xmx2g
Copy link
Member

@bmarty bmarty Oct 6, 2025

Choose a reason for hiding this comment

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

Please revert all the changes in this file. You can place your local configuration in the file local.properties instead, which is ignore by VCS.

Copy link
Author

@aliIsazadeh aliIsazadeh Oct 11, 2025

Choose a reason for hiding this comment

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

My apologies — I’m not sure when these changes were made and committed. It wasn’t intentional.

19 changes: 19 additions & 0 deletions libraries/mediaviewer/impl/src/main/AndroidManifest.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,19 @@
<?xml version="1.0" encoding="utf-8"?>
<!--
~ Copyright 2025 New Vector Ltd.
~
~ SPDX-License-Identifier: AGPL-3.0-only OR LicenseRef-Element-Commercial
~ Please see LICENSE files in the repository root for full details.
-->

<manifest xmlns:android="http://schemas.android.com/apk/res/android"
package="io.element.android.libraries.mediaviewer.impl">
<uses-permission android:name="android.permission.SYSTEM_ALERT_WINDOW" />

<application>
<service
android:name=".floatingvideo.FloatingVideoService"
android:enabled="true"
android:exported="false" />
</application>
</manifest>
Loading