Brogan Zumwalt [Inactive] Reporter
	Description • 10 years ago

Description:
When user is using browser media player to view or listen to a file that is longer than 1 hour, viewing the screen in portrait mode shows time elapsed and time remaining for media file very close together as though they were a single number.
   
Repro Steps:
1) Update a Flame device to BuildID: 20141117001201
2) Launch Browser in portrait mode
3) Navigate to media file URL that is longer than 1 hour (e.g. http://bit.ly/1wP6JtZ)
  
Actual:
Time elapsed and time remaining display close enough together to appear as one number.
  
Expected: 
There is sufficient space between the time elapsed and time remaining to tell the two numbers apart.
  
Environmental Variables:
Device: Flame 2.1 (319mb)(Kitkat Base)(Shallow Flash)
BuildID: 20141117001201
Gaia: 81160ad79e5b4c21967418dd63f1a1d08d77924e
Gecko: 3572aa3e6766
Version: 34.0 (2.1)
Firmware: V188-1
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0

Repro frequency: 3/3, 100%
See attached: screenshot & logcat

	Brogan Zumwalt [Inactive] Reporter
	Comment 1 • 10 years ago

Issue DOES occur on Flame 2.0 and Flame 2.2

Actual Results: Time elapsed and time remaining display close enough together to appear as one number.

Device: Flame 2.0 (319mb)(Kitkat Base)(Shallow Flash)
BuildID: 20141117000200
Gaia: 086a668942292168f312b3bb53e275fa0886fab1
Gecko: a57b299c5cf2
Version: 32.0 (2.0)
Firmware: V188-1
User Agent: Mozilla/5.0 (Mobile; rv:32.0) Gecko/32.0 Firefox/32.0

Device: Flame 2.2 (319mb)(Kitkat Base)(Shallow Flash)
Build ID: 20141117040203
Gaia: ddf5b92f43ec27c93ad4fea4fd1207da8936b8e7
Gecko: 21b745197618
Version: 36.0a1 (2.2)
Firmware Version: v188-1
User Agent: Mozilla/5.0 (Mobile; rv:36.0) Gecko/36.0 Firefox/36.0

	Derek Harris [:DerekH] - Dont need info. Last day 6/5/15
	Comment 3 • 10 years ago

[Blocking Requested - why for this release]:

The user is completely unable to read the time elapsed, or the time remaining this looks poor. Especially for users who would be listening to something like an audio book that is almost always more than an hour long.

	KTucker [:KTucker][Inactive 3/4/2016]
	Comment 4 • 10 years ago

[Blocking Requested - why for this release]:

	Gregor Wagner [:gwagner]
	Comment 5 • 10 years ago

Hema, is this your team or do you know where this bug belongs to?

	Hema Koka [:hema]
	Comment 6 • 10 years ago

assuming this is coming from common media controls -- may be dolske or roc would know.

	Justin Dolske [:Dolske]
	Comment 7 • 10 years ago

I don't know how FirefoxOS does videocontrols. If they are the default controls, the fix would be to make the touch controls hide elements when space-constrained. (For example, only show the current position but not the duration). We already do something like this for the desktop video controls.

	Robert O'Callahan (:roc) (email my personal email if necessary)
	Comment 8 • 10 years ago

I think wesj was at least partly responsible for the FxOS video controls.

Another option to fix this bug would be to reduce the font size of the time values when they exceed one hour.

	bhavana bajaj [:bajaj]
	Comment 9 • 10 years ago

Ken, can you help find the right owner here ? I know CJ owned this at some point but not sure who the right owner is at this point.

This is an ugly issue especially if you have long audio books on your phone.

	Ken Chang[:kenkai]
	Comment 10 • 10 years ago

Randy, please check this bug to see if you can take it.

	Randy Lin [:rlin] - Be Mozillian on 2015/01/26
	Comment 11 • 10 years ago

Hi Ken, 
Sorry, I can't help it because it may related to layout stuff.

	Bobby Chien
	Comment 12 • 10 years ago

hi peter, it looks like graphic or layout problem. could help to find someone's help?

	Peter Chang[:pchang]
	Comment 13 • 10 years ago

As mentioned in comment 8, I think it is related to the CSS configuration about video control. Clear the ni for graphic.

	Ken Chang[:kenkai]
	Comment 14 • 10 years ago

Hi Jet, do you have the idea of who can check this bug first? Thanks.

	howie [:howie]
	Comment 15 • 10 years ago

Seems like toolkit and video control issue, same owner with bug 1005031 ? ni Jared for more feedback, thanks.

	Jared Wein [:jaws] (please needinfo? me)
	Comment 16 • 10 years ago

Yeah, roc is right here that we just need to tweak the CSS for small sized videos.

This patch should fix it, but I don't have a mobile/touch build available. Wes, does this look good to you?

	Wesley Johnston (:wesj)
	Comment 17 • 10 years ago

Comment on attachment 8530052 [details] [diff] [review]
Patch

Review of attachment 8530052 [details] [diff] [review]:
-----------------------------------------------------------------

I think I would rather use some margin/padding (1em?) on the durationLabel here. Font size just lets the audio controls get even smaller, but the labels still run together.

	Jared Wein [:jaws] (please needinfo? me)
	Comment 18 • 9 years ago

Unassigning since I don't currently have an environment set up to test these changes. Hopefully someone who has an environment set up can take this bug. It should be a pretty quick fix, especially if you look at the patch that I put up to see where the changes need to be made.

	Steven Yang [:styang]
	Comment 19 • 9 years ago

Hi Ben, do you recommend anyone can help on this issue? thanks.

	Josh Cheng [:josh]
	Comment 20 • 9 years ago

Hi Ben,
Could you help? Thanks!

	Ben Francis [:benfrancis]
	Comment 21 • 9 years ago

This CSS code lives in B2G, Vivien & Fabrice are listed as module owners.

https://github.com/mozilla/gecko-dev/blob/master/b2g/chrome/content/touchcontrols.css

It looks like some JavaScript is manually resetting the width of the video element to 66% though, which is causing the compressed UI. Making the font smaller as in Jared's patch would work, although this CSS looks a little bit broken in general.

Vivien/Fabrice, any idea where this is being set in JavaScript?

	[:fabrice] Fabrice Desré
	Comment 22 • 9 years ago

The JS part is in the video control binding: http://mxr.mozilla.org/mozilla-central/source/toolkit/content/widgets/videocontrols.xml

touchcontrols.css is an override of videocontrols.css.

I haven't looked in detail, but http://mxr.mozilla.org/mozilla-central/source/toolkit/content/widgets/videocontrols.xml#357 matches your 66% resizing Ben!

	Josh Cheng [:josh]
	Comment 23 • 9 years ago

[Blocking Requested - why for this release]:
Hi Ben,
Could you help to check comment 22 from Fabrice?
Thanks!

	Mahendra Potharaju [:mahe]
	Comment 24 • 9 years ago

Nice to fix, not a blocker

	Mahendra Potharaju [:mahe]
	Comment 25 • 9 years ago

[Tracking Requested - why for this release]:

	Jared Wein [:jaws] (please needinfo? me)
	Comment 26 • 8 years ago

Closing old needinfo requests. This bug now only affects Firefox for Android (since B2G has been deprioritized), and in small video sizes.