Port |Bug 666306 - Video content should become large play button ...| to Modern

RESOLVED FIXED in seamonkey2.9

Status

SeaMonkey
Themes
--
enhancement
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: InvisibleSmiley, Assigned: InvisibleSmiley)

Tracking

Trunk
seamonkey2.9

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
Created attachment 592275 [details] [diff] [review]
patch [Checkin: Comment 3]

Bug 666306, which just landed on trunk, added a "click to play" button for HTML5 videos to Toolkit. Modern can reuse the artwork, but needs to have the CSS ported.
Attachment #592275 - Flags: review?(neil)

Comment 1

5 years ago
> +  skin/modern/global/media/clicktoplay-bgtexture.png               (/mozilla/toolkit/themes/winstripe/global/media/clicktoplay-bgtexture.png)
>    skin/modern/global/media/error.png                               (/mozilla/toolkit/themes/winstripe/global/media/error.png)
I think Neil prefers that we copy any images into Modern. In the past we've been caught out when toolkit/firefox changed the name of images or just removed them.

Comment 2

5 years ago
Comment on attachment 592275 [details] [diff] [review]
patch [Checkin: Comment 3]

>+  background-color: hsla(0,0%,10%,.5);
[hsla? whatever next?]
Attachment #592275 - Flags: review?(neil) → review+
(Assignee)

Comment 3

5 years ago
Comment on attachment 592275 [details] [diff] [review]
patch [Checkin: Comment 3]

http://hg.mozilla.org/comm-central/rev/c504e4a24ad9
Attachment #592275 - Attachment description: patch → patch [Checkin: Comment 3]
(Assignee)

Updated

5 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.9
You need to log in before you can comment on or make changes to this bug.