The documentation in the handbook on how to theme the flash player (http://drupal.org/node/34043) is very out of date and doesn't correlate to the current version of the audio module.

Since this page was last updated the 1pixelout flasher player has been included in the module and is the default. Now, with multiple players, the theme functions have changed as well.

So, let's update the documentation.

Some relevant info:
http://www.1pixelout.net/code/audio-player-wordpress-plugin/
http://www.innovatingtomorrow.net/2007/10/22/how-theme-1pixelout-flash-p...
http://musicplayer.sourceforge.net/

This is going in as part of the Google Highly Open Participation Contest. More info to follow.

CommentFileSizeAuthor
#10 audio_readme.patch1.46 KBkourge

Comments

mfer’s picture

mfer commented

GHOP issue: http://code.google.com/p/google-highly-open-participation-drupal/issues/...

Some more info:
http://www.innovatingtomorrow.net/2007/10/22/how-theme-1pixelout-flash-p...

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Title: Document How To Theme the Flash Players » GHOP #61: Document How To Theme the Flash Players
mfer’s picture

mfer commented
Status: Active » Postponed (maintainer needs more info)

Claimed by daryennaruto

mfer’s picture

mfer commented
Status: Postponed (maintainer needs more info) » Active
kourge’s picture

kourge commented
Assigned: Unassigned » kourge
kourge’s picture

kourge commented
Status: Active » Needs review

Please review the updated page.

mfer’s picture

mfer commented

@kourge - I'm in the middle of a vacation so I could only give this a quick once over. I'll take a better look at it in a couple days. Sorry for the delay.

Can you add the image diagraming the different locations that can be themed from the 1pixelout player? That is if the image is licensed properly. If you can't add images to the node attach it and I'll change the input filter for you.

kourge’s picture

kourge commented

@mfer
I've looked over the image (of the 1pixelout diagram) on Innovating Tomorrow and I'm not really sure what license it's under. I already have permission to change the input filter (to Documentation) and attach files, so no worries about that ;)

mfer’s picture

mfer commented

@kourge - I'm looking into the licensing. The image is distributed with the wordpress extension which is GPL licensed. But, on the 1pixelout.net the image is displayed on a Creative Commons licensed page.

In the meantime can you roll a patch for the README.txt file on this module the provides updated theming information. It can refer back to the handbook page. It's at http://cvs.drupal.org/viewvc.py/drupal/contributions/modules/audio/READM...

kourge’s picture

kourge commented
StatusFileSize
new audio_readme.patch1.46 KB

I'm assuming the image is licensed under CC by 2.0 because
1) It appears on the website, which is licenses all content under CC by 2.0 and
2) GPL is meant for code, not content like images (just like CC is meant for content, not code)
So I've put the image onto the handbook page.

For README.txt, patch attached. It fixes a few punctuation and it moves $Id$ to the top.

mfer’s picture

mfer commented

This looks good to me but I sent it off to a designer/themer type (non-coder) to review.

drewish’s picture

drewish commented
Status: Needs review » Active

I committed the patches to DRUPAL-5 and HEAD. I'm going to leave this open until mfer signs off on the handbook pages.

mfer’s picture

mfer commented
Status: Active » Closed (fixed)

The handbook page looks good and the patch is committed. Marking this as closed.

@kourge - Thanks for working on this.

mfer’s picture

mfer commented
Status: Closed (fixed) » Fixed

I meant fixed. :)

kourge’s picture

kourge commented

Yay!

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.