Context Navigation

← Previous Ticket
Next Ticket →

#23821 closed enhancement (fixed)

Twenty Thirteen: Add styles for mediaelement players

Reported by:	obenland	Owned by:	lancewillett
Priority:	normal	Milestone:	3.6
Component:	Bundled Theme	Version:	3.6
Severity:	normal	Keywords:	has-patch
Cc:		Focuses:

Description

As the title says.

Add support for the audio shortcode and make it look great.

Proposed patch still needs to be run by Joen and a ton of testing.

Attachments (5)

23821.diff (1.4 KB ) - added by obenland 13 years ago.
23821.1.diff (2.6 KB ) - added by obenland 13 years ago.
flipped-icon.png (17.8 KB ) - added by obenland 13 years ago.
styled-players.png (10.7 KB ) - added by obenland 13 years ago.
23821.2.diff (389 bytes ) - added by obenland 13 years ago.

Download all attachments as: .zip

Change History (12)

@obenland
13 years ago

Attachment 23821.diff added

#1 @DrewAPicture
13 years ago

23821.diff definitely looks better.

Before: http://cl.ly/image/2J283M2X3w0y

After: http://cl.ly/image/431K3m1R111R

#2 @obenland
13 years ago

Summary Twenty Thirteen: Make audio post formats even more awesome → Twenty Thirteen: Add styles for mediaelement players

@obenland
13 years ago

Attachment 23821.1.diff added

@obenland
13 years ago

Attachment flipped-icon.png added

@obenland
13 years ago

Attachment styled-players.png added

#3 @obenland
13 years ago

Added styles for the audio and video player based on Joen's feedback: styled-players.png .

Also flipped the audio icon for RTL: flipped-icon.png

Last edited 13 years ago by obenland (previous) (diff)

#4 @lancewillett
13 years ago

Owner set to lancewillett
Resolution → fixed
Status new → closed

In 23781:

Twenty Thirteen: style up the new mediaelement audio and video players. Props obenland, fixes #23821.

#5 @lancewillett
13 years ago

Keywords needs-refresh added; needs-testing removed
Resolution fixed
Status closed → reopened

@obenland Can you take another look at this, for the video player? Black background, or the same dark color the audio player uses.

@obenland
13 years ago

Attachment 23821.2.diff added

#6 @obenland
13 years ago

Keywords needs-refresh removed

#7 @lancewillett
13 years ago

Resolution → fixed
Status reopened → closed

In 23835:

Twenty Thirteen: style native mediaelement video player with a black background color. Props obenland, closes #23821.

Note: See TracTickets for help on using tickets.

Download in other formats:

Make WordPress Core

Context Navigation

#23821 closed enhancement (fixed)

Twenty Thirteen: Add styles for mediaelement players

Description

Attachments (5)

Change History (12)

@obenland 13 years ago

#1 @DrewAPicture 13 years ago

#2 @obenland 13 years ago

@obenland 13 years ago

@obenland 13 years ago

@obenland 13 years ago

#3 @obenland 13 years ago

#4 @lancewillett 13 years ago

#5 @lancewillett 13 years ago

@obenland 13 years ago

#6 @obenland 13 years ago

#7 @lancewillett 13 years ago

Download in other formats:

@obenland
13 years ago

#1 @DrewAPicture
13 years ago

#2 @obenland
13 years ago

@obenland
13 years ago

@obenland
13 years ago

@obenland
13 years ago

#3 @obenland
13 years ago

#4 @lancewillett
13 years ago

#5 @lancewillett
13 years ago

@obenland
13 years ago

#6 @obenland
13 years ago

#7 @lancewillett
13 years ago