jqueryrotate
jqueryrotate copied to clipboard
Throws error when page contains no 'HEAD' element
What steps will reproduce the problem?
1. Right-Click any image on a webpage
2. Choose 'Open image in new tab'
What is the expected output? What do you see instead?
Plug-in should load normally,
Instead shows error found, 'No property "style" of undefined'
What version of the product are you using? On what operating system?
v2.3, in Chrome 29.0.1547.66, on Win7 64bit
Please provide any additional information below.
May not be fixable since the information you get from 'head' is needed to
detect what browser is being used. But I thought I'd mention it anyways.
I got the script to run perfectly for my needs by just setting the vars to what
would have been detected for Chrome.
Thank you for the wonderful plugin!
Original issue reported on code.google.com by [email protected]
on 10 Sep 2013 at 11:15
Well I tried to reproduce on mine chromium but without any succes. Do you have
any page where I can see the problem live ?
Thanks in advance
Original comment by [email protected]
on 17 Sep 2013 at 11:16
Unfortunately I have no live examples. I wrote a little script for my own use
and run it through Tampermonkey.
Here is a link my script would run on though -
http://lebojct.com/Swift-N-Sure/5-11-13-CarWash/images/01.jpg
If you inspect the page using Chrome you see -
<html>
<body style="margin: 0px;">
<img style="-webkit-user-select: none" src="http://lebojct.com/Swift-N-Sure/5-11-13-CarWash/images/01.jpg">
</body>
</html>
There is a <BODY> element, but there is not a <HEAD> element.
So the plugin breaks around line 10 running the code -
styles = document.getElementsByTagName("head")[0].style
There is no <head> element, so document.getElementsByTagName("head") returns
null, and then the plugin stops when trying to execute .style on null.
Original comment by [email protected]
on 17 Sep 2013 at 10:38
Ahh hmm ok now i get it :) Still it's weird as a <head></head> element should
be created internally in browser even if not defined explicite, I need to
investigate a bit more. Thanks for finding
Original comment by [email protected]
on 18 Sep 2013 at 7:37
Ahh I get it, you're using it as/trough extension - in that case a normal
global context might be slighty different. Still there should be other,easy way
to append styles crossbrowse mhmm..
Original comment by [email protected]
on 18 Sep 2013 at 7:39