Make WordPress Core

Opened 12 years ago

Closed 11 years ago

#25224 closed defect (bug) (duplicate)

Video controls not showing

Reported by: kyleziegler's profile kyleziegler Owned by:
Milestone: Priority: normal
Severity: normal Version: 3.6
Component: External Libraries Keywords:
Focuses: Cc:

Description

The video control icons do not show on the videos. Specifically the small play button (bottom left), large play button (center), and the volume and maximize controls located bottom right. See my attached images for clarification.

You will see this problem in every browser as the wrong file is being referenced.

Patch: Change the SVG images to PNG within the mediaelementplayer.min.css

Attachments (3)

After.PNG (416.4 KB) - added by kyleziegler 12 years ago.
After Patch
Before.PNG (441.3 KB) - added by kyleziegler 12 years ago.
Before Patch
FilesToChange.PNG (16.3 KB) - added by kyleziegler 12 years ago.
Files to change

Download all attachments as: .zip

Change History (17)

@kyleziegler
12 years ago

After Patch

@kyleziegler
12 years ago

Before Patch

@kyleziegler
12 years ago

Files to change

#1 @Ipstenu
12 years ago

Cannot reproduce on 3.6 with Chrome.

You're looking at the about page, right?

http://cl.ly/image/0a2U3t3O0S3u

Looks like it should. So do all embeds on my sites.

#2 @Ipstenu
12 years ago

  • Keywords reporter-feedback added

#3 @SergeyBiryukov
12 years ago

  • Component changed from Embeds to External Libraries

Could not reproduce either. Tested in Firefox 23 (Windows and Mac OS X), Chrome 29 (Windows and Mac OS X), IE 8 (Windows), Opera 12 (Mac OS X), Opera 15 (Windows).

#4 @kyleziegler
12 years ago

@ipstenu Yes, I was looking at the about page regarding the 3.6 release.

That's odd that both of you cannot replicate the issue, my company wouldn't have changed that file so I figured it was from a Wordpress update. We updated to 3.6 the other week, we should get some more people to test this though.

#5 @Ipstenu
12 years ago

What browser/OS is this happening on?

#7 @kyleziegler
12 years ago

@Ipstenu I was on W7, I personally tested FF 23, IE 10, and Chrome 29.x. We also tested it on FF23 and Safari 6.x for OSX 10.8

@ocean90 I took a glance at that post. I should've tested the issue in IE8 and see if it worked, I could undo my patch and test that if it's necessary?

Side notes: We own our servers

#8 @SergeyBiryukov
12 years ago

What happens if you open those .svg files in a browser directly? Does adding the image/svg+xml mime type to .htaccess (as described in the topic) fix it?

#9 follow-up: @kyleziegler
12 years ago

@SergeyBiryukov I believe that .svg is a defined file type on our servers. Although I'll still need to undo the patch and test that. Could anyone remove the image/svg+xml from their .htaccess file and see if they get the same issue as us?

#10 in reply to: ↑ 9 @SergeyBiryukov
12 years ago

Replying to kyleziegler:

Could anyone remove the image/svg+xml from their .htaccess file and see if they get the same issue as us?

I don't have image/svg+xml in my .htaccess file, however I've found it in conf/mime.types, commented it out and restarted the server. bigplay.svg and controls.svg were then served as text/html, and I was able to reproduce the issue.

Looks like MediaElement uses .svg files for Retina displays:
https://github.com/johndyer/mediaelement/commit/e3ab5e18b4f3755de7b0f1c339c9f462b3be59f4

This appears to be a server configuration issue, not sure if we can do anything here, or should report it upstream.

#11 @kyleziegler
12 years ago

@SergeyBiryukov That's great I was hoping I wasn't the only one with the issue. Although like someone else noted, most individuals aren't in a position to add mime types to their server or edit the MediaElement style sheets.

#12 @SergeyBiryukov
12 years ago

  • Keywords reporter-feedback removed

#14 @johnbillion
11 years ago

  • Keywords has-patch removed
  • Milestone Awaiting Review deleted
  • Resolution set to duplicate
  • Status changed from new to closed

This is the same issue as #26591

Note: See TracTickets for help on using tickets.