Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use of pseudo-tag in hierarchical selector causes exception #6

Closed
GoogleCodeExporter opened this issue Mar 11, 2016 · 14 comments
Closed

Comments

@GoogleCodeExporter
Copy link

What steps will reproduce the problem?
1. Take one of the IE7-JS test pages, such as
http://ie7-js.googlecode.com/svn/test/child.html
2. Add the following CSS to the style section:

#test a:hover
{
    background-position: 0px -123px;
}

3. Open the page in IE6.

What is the expected output? What do you see instead?
As these is no body element with the id 'test', nothing should be seen.
However, there should also be no issue in loading the page.

Instead, a JavaScript exception is thrown. Debugging results in the
following problem being highlighted:

Line 529:
var _selectorFunction=function(e0,s){IE7._indexed++;var
r=[],p={},reg=[],d=document;var
e1=IE7._byId(d,'test');if(e1&&e0==d||e0.contains(e1)){var
p2=0,i2,e2,n2=e1.getElementsByTagName('A');for(i2=0;(e2=n2[i2]);i2++){r[r.length
]=e2;if(s)return
e2;}}return s?null:r}

Object doesn't support this property or method on:
if(e1&&e0==d||e0.contains(e1))

What version of the product are you using? On what operating system?
2.0(beta) of IE7-JS, on IE6.0.2900, on Windows XP.

Original issue reported on code.google.com by arcane%m...@gtempaccount.com on 8 Jan 2008 at 6:06

@GoogleCodeExporter
Copy link
Author

> if(e1&&e0==d||e0.contains(e1))

I can see the problem. Will fix for the next release.

Original comment by dean.edw...@gmail.com on 8 Jan 2008 at 4:26

  • Changed state: Accepted

@GoogleCodeExporter
Copy link
Author

This bug is pretty much a dealbreaker for our site, removing all teh :hover and
:before isn't much of an option. Really excited to try out the next release 
though :)

Original comment by simianuprising on 11 Jan 2008 at 9:21

@GoogleCodeExporter
Copy link
Author

We're currently experiencing an exception on the same line of code. You've 
probably
grown tired of answering this question but is there any note on when the next 
release
is due Dean? We're currently using your IE7 js for our bachelor degree 
assignment,
and we'd really appreciate it if we were to know if we're able to count on a 
next
release that'll fix this bug in time (which is pretty much around the end of 
this week).

Thanks in advance, and keep up the good work!

Original comment by ninh....@gmail.com on 14 Jan 2008 at 11:22

@GoogleCodeExporter
Copy link
Author

I'll issue a new release sometime in the next few days. I can't make a release 
every
time a fix a bug so there will be at least a two week delay between releases. 
That
said, I've fixed a few bugs since beta 1 and will release beta 2 shortly.

I could run this project much like I run base2. In base2 I have a "live" 
directory
which contains the most up-to-date version of the library. Shall I do the same 
for IE7?

Original comment by dean.edw...@gmail.com on 14 Jan 2008 at 11:41

  • Changed state: Started

@GoogleCodeExporter
Copy link
Author

That would be awesome! :-)

Original comment by ninh....@gmail.com on 15 Jan 2008 at 3:35

@GoogleCodeExporter
Copy link
Author

OK. The dev area is:

http://ie7-js.googlecode.com/svn/trunk/lib/

You can link to the source files there for the latest development version. 
Currently
though it is still the beta 1 version. I will update it as soon as I have time 
to
perform some basic testing.

Original comment by dean.edw...@gmail.com on 15 Jan 2008 at 8:25

@GoogleCodeExporter
Copy link
Author

Hi Dean, again, thanks for the fast response Dean and for allowing us to access 
your
version control trunk!

Original comment by ninh....@gmail.com on 16 Jan 2008 at 10:04

@GoogleCodeExporter
Copy link
Author

Hello Dean, great work and I'm waiting for new bug-fix release too :).

Original comment by aTom...@gmail.com on 21 Jan 2008 at 4:58

@GoogleCodeExporter
Copy link
Author

Hi Dean,
thanks a lot for your improved IE7 and IE8 scripts.
I was about to start writing one myself, to fix the couple most annoying bugs 
in IE7
(the actual browser) when I noticed you had posted your new versions!

I'm also holding my breath for Google Code revision 20, as this hierarchical 
selector
bug is a show-stopper for me.

Original comment by etat...@gmail.com on 23 Jan 2008 at 12:08

@GoogleCodeExporter
Copy link
Author

Thanks for the dev links - the amount that you rock is immeasurable.

Michael Fouquette
MikeWorks.NET

Original comment by michael....@gmail.com on 25 Jan 2008 at 8:53

@GoogleCodeExporter
Copy link
Author

Original comment by dean.edw...@gmail.com on 4 Feb 2008 at 6:28

  • Changed state: Fixed

@GoogleCodeExporter
Copy link
Author

Issue 13 has been merged into this issue.

Original comment by dean.edw...@gmail.com on 11 Feb 2010 at 6:29

@GoogleCodeExporter
Copy link
Author

Issue 15 has been merged into this issue.

Original comment by dean.edw...@gmail.com on 11 Feb 2010 at 6:29

@GoogleCodeExporter
Copy link
Author

Issue 24 has been merged into this issue.

Original comment by dean.edw...@gmail.com on 11 Feb 2010 at 6:30

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant