tel8618217223380 / gwtquery

Automatically exported from code.google.com/p/gwtquery
MIT License
0 stars 0 forks source link

cant do simple #ID select? #105

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
I know I must be insane... but I'm using gwtquery-1.1.0.jar and if I do this:

GQuery.$("<div><div id=\\"name\\">blah</div></div>").find("#name")

it returns nothing, but if I do this:

GQuery.$("<div><div id=\\"name\\">blah</div></div>").find("[id=name]")

it returns "<div id="name">blah</div>" as expected

tracing thru it is getting to SelectorEngineNative, and because the query 
starts with hash(#) it is then getting directed towards  
impl/SelectorEngine.veryQuickId()... which is returning nothing

if I do an expression of "impl.select(selector, ctx)" in eclipse (the 
alternative code path for veryQuickId)... then the "#name" query seems to work

surely I'm doing something wrong, or have something setup incorrectly?   the 
#id queries in my app used to work fine...  and its such a fundamental query 
that the regular tests would have picked it up by now?

am I missing any other info you would need to help explain?

Original issue reported on code.google.com by n...@batterii.com on 28 Sep 2011 at 5:27

GoogleCodeExporter commented 9 years ago

Original comment by julien.d...@gmail.com on 5 Oct 2011 at 8:36

GoogleCodeExporter commented 9 years ago
It should be fixed in 1.1.0, could you check it?

Original comment by manuel.carrasco.m on 25 Oct 2011 at 9:03

GoogleCodeExporter commented 9 years ago
isn't that what I said I was using?  "...but I'm using gwtquery-1.1.0.jar and 
if I do this..."

Original comment by n...@batterii.com on 25 Oct 2011 at 3:34

GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
You was using a snapshot version of GwtQuery 1.1.0. GwtQuery 1.1.0 was released 
a few days ago and a lot of changes was made since you open the bug...

And for your info, this bug should be fixed by the revision r854

Original comment by julien.d...@gmail.com on 25 Oct 2011 at 3:43

GoogleCodeExporter commented 9 years ago
my apologies.

yes, this looks to be fixed now.  thanks!

Original comment by n...@batterii.com on 25 Oct 2011 at 5:36

GoogleCodeExporter commented 9 years ago

Original comment by manuel.carrasco.m on 31 Oct 2011 at 2:32