When a videoJS-enabled node is added to a panel, the panels cogs disappear for users in IE8.

This may be a conflict with admin module, as the JS error seems to be in jquery.drilldown.js

I haven't had time to investigate too far - the attached patch fixes it, but seems a bit hacky to me...

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Jorrit’s picture

Status: Active » Postponed (maintainer needs more info)

Can you post a screenshot of the javascript error and the page on which the error occurs?

Jorrit’s picture

Status: Postponed (maintainer needs more info) » Closed (cannot reproduce)

I can't reproduce this.

malcomio’s picture

Status: Closed (cannot reproduce) » Active
FileSize
185.46 KB
202.27 KB

I've just done some testing on a very basic environment, in the stark theme, and it is definitely a conflict with the admin module - with that module disabled there's no problem.

The IE debugger error is:

SCRIPT5007: Unable to get value of the property 'attr': object is null or undefined
jquery.drilldown.js?U, line 87 character 15

Jorrit’s picture

It looks like that site is a test site. Can I get administrator access to investigate the problem?

malcomio’s picture

yes thanks - i'll send you a direct message with the details

Jorrit’s picture

Status: Active » Closed (works as designed)

First of all, I don't understand why you post this issue for the Video.js module. This is not a bug or incompatibility between Video.js and any other module. It is actually a bug in the Admin module, which has already been reported in #969916: jquery.drilldown.js Breadcrumb error. You can try the patches there. The Admin module has not seen a new release in 1.5 years, perhaps you should try a module that is maintained better.

If I am wrong and this actually is a Video.js bug, please let me know. The source of the problem that I found is that in jquery.drilldown.js there is a for loop over all properties of a project, but it does not exclude standard properties like indexOf. IE8 fails on that.

malcomio’s picture

Thanks for your help - I hadn't found that issue, and admin had been working fine until we came across panel pages with videos in, so I'd assumed the problem was likely to be at the video end.

I'll try the patches from that issue if I get time - in the short term my patch above sidesteps the problem.