Closed
Bug 489806
Opened 17 years ago
Closed 17 years ago
Implement {SCREENCAST} TikiWiki plugin
Categories
(support.mozilla.org :: General, defect)
support.mozilla.org
General
Tracking
(Not tracked)
VERIFIED
FIXED
1.1
People
(Reporter: laura, Assigned: ecooper)
References
Details
(Whiteboard: sumo_only screencasts)
Attachments
(2 files, 1 obsolete file)
|
4.00 KB,
patch
|
laura
:
review+
|
Details | Diff | Splinter Review |
|
471.22 KB,
image/png
|
Details |
As per https://wiki.mozilla.org/Support/ScreencastsPRD, to be used for embedding a screencast in a wiki page.
| Assignee | ||
Comment 1•17 years ago
|
||
Just dropping this in the bug for reference.
| Assignee | ||
Comment 2•17 years ago
|
||
Syntax:
{SCREENCAST(file=>fileId)}Watch this video or whatever text{SCREENCAST}
'file' is the file ID of the screencast. Params for sizes aren't available yet simply because our current pool of screencasts can't be resized. As we slowly move away from the jing swf files, we can implement size parameters. Though, I wonder how useful they are if we're always going to be showing screencasts via an overlay pop-up.
"Watch this video or whatever text" is optional. If it's not included or empty, the default message defined in the preferences panel will be used.
All the plugin really does is set the stage for the javascript launcher for the player.
Attachment #376706 -
Attachment is obsolete: true
Attachment #378604 -
Flags: review?(paul.craciunoiu)
Attachment #378604 -
Flags: review?(laura)
Comment 3•17 years ago
|
||
Need to review this properly, but I opened the file in my browser and noticed there's an extra ">" at the end, last line. Probably unintended?
Not sure if it's a problem. Just making a note so we don't forget :)
| Reporter | ||
Updated•17 years ago
|
Attachment #378604 -
Flags: review?(laura) → review+
| Assignee | ||
Comment 4•17 years ago
|
||
Attachment #378604 -
Flags: review?(paul.craciunoiu)
| Assignee | ||
Updated•17 years ago
|
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Comment 5•17 years ago
|
||
Comment 6•17 years ago
|
||
Verified FIXED -- testing is going well, and the screenshot in comment 5 shows that the override text works.
Status: RESOLVED → VERIFIED
Updated•16 years ago
|
Whiteboard: sumo_only screencasts
You need to log in
before you can comment on or make changes to this bug.
Description
•