When including the jPlayer skin via $element['#attached']['css'], the CSS for the skin is not applied.

You can use an alternative method of inclusion as a workaround:

  • Include the CSS via a 'style' tag. The downside to this is that the image locations in the CSS file must be absolute.
  • Include the CSS via the drupal_add_html_head function. The downside to this is that the placement of the repeat button is altered due to interference from other Drupal CSS. This is the method the current build uses.
CommentFileSizeAuthor
#1 npp-jplayer-repeat-button-misplacement.png84.34 KBmusicalvegan0

Comments

musicalvegan0’s picture

musicalvegan0 commented
StatusFileSize
new NPP jPlayer Repeat Button Misplacement Example84.34 KB

See attached screenshot to see the misplacement of the repeat button.

musicalvegan0’s picture

musicalvegan0 commented
Version: » 7.x-1.0
Status: Active » Postponed (maintainer needs more info)

To help alleviate this problem, I've made displaying the repeat button for jPlayer optional with this commit: db0f820253234012b37c78909f14b4841d349ce7

I'm going to mark this as postponed and wait to see if anyone else complains about this, in which case I will begin work on it.

musicalvegan0’s picture

musicalvegan0 commented
Issue summary: View changes

Added that the current build's method for adding skin CSS to jPlayer.

musicalvegan0’s picture

musicalvegan0 commented
Issue summary: View changes

Closing this issue due to its age and irrelevancy. This issue seemed to be solved with later version of jPlayer.

musicalvegan0’s picture

musicalvegan0 commented
Status: Postponed (maintainer needs more info) » Closed (fixed)