Closed
Bug 721881
Opened 13 years ago
Closed 13 years ago
Port |Bug 666306 - Video content should become large play button ...| to Modern
Categories
(SeaMonkey :: Themes, enhancement)
SeaMonkey
Themes
Tracking
(Not tracked)
RESOLVED
FIXED
seamonkey2.9
People
(Reporter: InvisibleSmiley, Assigned: InvisibleSmiley)
Details
Attachments
(1 file)
4.89 KB,
patch
|
neil
:
review+
|
Details | Diff | Splinter Review |
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•13 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•13 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•13 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•13 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.9
You need to log in
before you can comment on or make changes to this bug.
Description
•