Please provide a custom field, which calls for example theme('opvideo_custom',$video);.
It's hard to theme a video field now (you use _op_video_render_* instead of theme functions).

Comments

jbrown’s picture

jbrown commented
Assigned: Unassigned » jbrown
Category: feature » support

There are two theme functions which you can override:


function theme_op_video_preview_image($image) {
  return $image;
}


function theme_op_video_player($player) {
  return $player;
}
alippai’s picture

alippai commented

I don't want to use the existing $player or $image, because I want to use my own players. I want to get $video, to theme it for my player.

jbrown’s picture

jbrown commented

In that case - you need to hack player.inc. We'll probably move op_video to flowplayer eventually.

alippai’s picture

alippai commented
Category: support » feature

I know, that I have to hack player.inc - that is the reason why I set this issue as feature request :)
About that change of the video player: a pluggable system would be nice - in this case we should change the player at views formatter settings and add custom players via hooks. IMHO this is the right way...
The player plugin needs to define the players/themes (set if the player/image etc. can be a link or only it can be a link destination, or it can be rendered only without link) and at formatters we should have to choose from them (a rendered object->destination pair).

jbrown’s picture

jbrown commented
Status: Active » Closed (fixed)