Currently the only way to theme the Flash code is by using string manipulation in theme_embed.
This patch provides another theme function where the actual Flash Code is assembled, allowing you to change the parameters and attributes of the outputted HTML code in a theme function override.

CommentFileSizeAuthor
flashtheme.patch3.32 KBdanielb

Comments

alex_b’s picture

alex_b commented
Status: Needs review » Needs work

Great thing.

--- ./slideshowpro.module 2009-02-27 11:14:48.000000000 +1000
+++ ../slideshowpro_patched/slideshowpro.module 2009-02-27 11:16:32.000000000 +1000

If you clean up the patch I'll commit :-) Are you not creating the patch with cvs diff?

We should also clean up theme_slideshowpro_feed() at some point. There is lots of application logic in the theme function. At least we should make it a template.

danielb’s picture

danielb commented

What do you mean clean up the patch? I am using SVN. I don't really know the best way to set up this sort of patch, so I just made two directories, 'slideshowpro' and 'slideshowpro_patched' and then did a diff. I've tried applying the patch and it works without changing the names of anything.
Can you tell me the technique you would use to change 2 files with one patch, as in how would you set up your files to run the diff?