Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

Picture-in-picture mode #2818

Closed
Lululalu opened this issue May 24, 2019 · 15 comments
Closed

Picture-in-picture mode #2818

Lululalu opened this issue May 24, 2019 · 15 comments
Assignees
Labels
E2 Estimation Point: easy, half a day to 2 days eng:qa:verified QA Verified eng:ready Ready for engineering Feature:Media needs:ac Needs Android Component Work

Comments

@Lululalu
Copy link

Lululalu commented May 24, 2019

Why/User Benefit/User Problem

Android 8.0 allows videos to played floating on the screen. This feature gives a better multi-task and viewing experience.

What/Requirements

Adding PiP support to videos.

Acceptance Criteria (how do I know when I’m done?)

When it's possible to watch a video outside the app thanks to pip support.


See eng notes below: #2818 (comment)

How it currently looks on Reference Browser:

CelebratedPepperyKitty-sizeUNITO-UNDERSCORE!restricted!

┆Issue is synchronized with this Jira Task

@Lululalu Lululalu added the feature request 🌟 New functionality and improvements label May 24, 2019
@suoko
Copy link

suoko commented May 24, 2019

Samsung internet browser has this feature, a quite nice feature.

@vrudesai vrudesai added Feature:Media and removed feature request 🌟 New functionality and improvements labels May 28, 2019
@vrudesai vrudesai added this to the Post-MVP Backlog milestone May 28, 2019
@jonalmeida jonalmeida changed the title Picture-in-picture support As a user, I want to play videos in Picture-in-picture mode Jul 22, 2019
@jonalmeida jonalmeida added the feature request 🌟 New functionality and improvements label Jul 22, 2019
@jonalmeida
Copy link
Contributor

jonalmeida commented Jul 22, 2019

@Lululalu I hope you don't mind me hijacking your feature request so I could give it more context. :)

@Lululalu
Copy link
Author

@jonalmeida it's OK. Do what's best.

@vesta0 vesta0 modified the milestone: Feature Backlog Jul 26, 2019
@vesta0 vesta0 changed the title As a user, I want to play videos in Picture-in-picture mode Picture-in-picture mode Jul 29, 2019
@vesta0 vesta0 mentioned this issue Jul 29, 2019
7 tasks
@Poopooracoocoo
Copy link

I'd love a custom implementation like NewPipe which gives you PiP on versions before O. Samsung's video player doesn't require O either.

@vesta0 vesta0 added could and removed feature request 🌟 New functionality and improvements labels Sep 19, 2019
@boek boek added the E2 Estimation Point: easy, half a day to 2 days label Sep 23, 2019
@lime124 lime124 added ux:xs and removed ux:xs labels Sep 25, 2019
@AmyYLee
Copy link
Collaborator

AmyYLee commented Nov 4, 2019

@lime124 I took a look at the reference browser PIP and it seems UX is pretty minimal. Is there any UX requirement for this? I can take it if there is.

@jonalmeida
Copy link
Contributor

jonalmeida commented Nov 4, 2019

I took a look at the reference browser PIP and it seems UX is pretty minimal.

One use case to consider: If a video is in PIP-mode, should you be able to browse other content at the same time? e.g. play a video in PIP and read an article on a different tab. In the reference browser implementation, you can only watch the video and use other apps, not R-B itself.

@lime124
Copy link
Collaborator

lime124 commented Nov 12, 2019

sorry for being slow @AmyYLee - i agree the it should be a minimal effort. if i recall it's mostly reviewing the implementation. @jonalmeida calls out a good use case to consider and make a decision on. feel free to pick this up amy

@AmyYLee
Copy link
Collaborator

AmyYLee commented Nov 12, 2019

I took a look at the reference browser PIP and it seems UX is pretty minimal.

One use case to consider: If a video is in PIP-mode, should you be able to browse other content at the same time? e.g. play a video in PIP and read an article on a different tab. In the reference browser implementation, you can only watch the video and use other apps, not R-B itself.

Hi @jonalmeida if it's possible to play a video in PIP while being able to go to a different tab we should do this. We are able to do this in desktop and I don't see any negatives if we allow this on Fenix.

@ekager
Copy link
Contributor

ekager commented Apr 9, 2020

This is landed behind a Nightly/Debug feature flag for now and should be working for custom tabs and normal browsing.

@sv-sdeiac
Copy link

Hi, I can confirm that the PiP mode is working using a Firefox Preview Debug build from master 4/13 GV: 77.0a1.

Devices:

  • Google Pixel 3 (Android 10);
  • Samsung Galaxy Note 8 (Android 9);
  • Xiaomi Mi 8 Lite (Android 9);
  • LG g7 fit (Android 8.1).

Additionally, performing the verification I found 3 new bugs related to the feature (9885, 9910, 9911).

@sv-sdeiac sv-sdeiac added eng:qa:verified QA Verified and removed eng:qa:needed QA Needed labels Apr 14, 2020
ekager added a commit to ekager/fenix that referenced this issue Apr 22, 2020
ekager added a commit to ekager/fenix that referenced this issue Apr 22, 2020
ekager added a commit to ekager/fenix that referenced this issue Apr 22, 2020
ekager added a commit to ekager/fenix that referenced this issue Apr 22, 2020
ekager added a commit that referenced this issue Apr 23, 2020
Amejia481 pushed a commit to Amejia481/fenix that referenced this issue Apr 23, 2020
@liuche liuche mentioned this issue Apr 28, 2020
32 tasks
@data-sync-user data-sync-user changed the title Picture-in-picture mode FNX2-18303 ⁃ Picture-in-picture mode Aug 5, 2020
@data-sync-user data-sync-user changed the title FNX2-18303 ⁃ Picture-in-picture mode FNX3-16602 ⁃ Picture-in-picture mode Aug 11, 2020
@data-sync-user data-sync-user changed the title FNX3-16602 ⁃ Picture-in-picture mode Picture-in-picture mode May 20, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
E2 Estimation Point: easy, half a day to 2 days eng:qa:verified QA Verified eng:ready Ready for engineering Feature:Media needs:ac Needs Android Component Work
Projects
None yet
Development

No branches or pull requests