Opened 7 years ago

Closed 7 years ago

Last modified 6 years ago

#3933 closed defect (fixed)

Dark background behind embedded media player in Browse

Reported by: greenfeld Owned by: manuq
Priority: Low Milestone:
Component: Browse Version: Unspecified
Severity: Unspecified Keywords: regression, olpc-test-pending
Cc: humitos Distribution/OS: OLPC
Bug Status: Unconfirmed

Description

There is a dark background behind the embedded media player in Browse in OLPC 13.1.0 os2.

While many media players like dark backgrounds, the embedded player shows progress information with black text, making it hard to read.

Seen with the OLPC library samples included with the build.

Attachments (4)

Browse.129.1.png (34.4 KB) - added by humitos 7 years ago.
Browse.140.png (22.4 KB) - added by humitos 7 years ago.
140 on 13.1.0.png (22.1 KB) - added by humitos 7 years ago.
Browse v140 running on 13.1.0 playing an ogg file
140 on 12.1.0.png (20.8 KB) - added by humitos 7 years ago.
Browse v140 running on 12.1.0 playing an ogg file

Download all attachments as: .zip

Change History (16)

comment:1 Changed 7 years ago by dsd

  • Milestone changed from Unspecified by Release Team to 0.98
  • Owner changed from erikos to dsd
  • Status changed from new to assigned

comment:2 Changed 7 years ago by godiard

  • Priority changed from Unspecified by Maintainer to Low

comment:3 Changed 7 years ago by humitos

  • Cc humitos added

How it should work?

I've just test the Gecko version of Browse (129.1) with an .ogg file and it show a visualization plugin with a progress bar at the bottom, a pause button and a volume slider.

The version included in 12.1.0 build 21 (140) shows a dark background as well, but not too dark to make difficult to see the progress bar.

What do we want?

Changed 7 years ago by humitos

Changed 7 years ago by humitos

comment:4 follow-up: Changed 7 years ago by dsd

Now check 13.1.0 and you will see what this bug report is about.

comment:5 in reply to: ↑ 4 Changed 7 years ago by humitos

Replying to dsd:

Now check 13.1.0 and you will see what this bug report is about.

Yes, yes. I've already checked that and I agree with the bug report that greenfeld filled here.

The thing is: how do we want to fix it?

  • show the visualization as it was in the Gecko version or
  • make the background as light as it is in 12.1.0

comment:6 Changed 7 years ago by dsd

Making it use the totem plugin for playback is an option that seems unlikely to happen, so fixing the appearance of the internal ogg player that is already appearing seems like the way to go.

Changed 7 years ago by humitos

Browse v140 running on 13.1.0 playing an ogg file

Changed 7 years ago by humitos

Browse v140 running on 12.1.0 playing an ogg file

comment:7 Changed 7 years ago by humitos

I tried what manuq told me about the sugar-artwork and the entry BrowseTabPage { background-color: @black; } but it didn't work. I changed it to @white

I ran Browse v140 on 13.1.0 and I noticed that the difference is not the webpage / browse background but the plugin background (check the differences in the screenshots)

comment:8 Changed 7 years ago by humitos

  • Keywords regression added

comment:10 Changed 7 years ago by manuq

  • Owner changed from dsd to manuq

comment:11 Changed 7 years ago by manuq

  • Keywords olpc-test-pending added
  • Resolution set to fixed
  • Status changed from assigned to closed

Pushed 4bc02188 .

comment:12 Changed 6 years ago by dnarvaez

  • Milestone 0.98 deleted

Milestone 0.98 deleted

Note: See TracTickets for help on using tickets.