"input:visible" selector does not work properly #11

Closed
Arithmomaniac opened this Issue Jun 21, 2012 · 2 comments

Comments

Projects
None yet
2 participants
@Arithmomaniac

In my C# program, it fails to exclude those with a type="hidden" attribute.
I do not have this problem with "select:visible" queries - it excludes them as per JQuery spec.
Running 1.1.1 through NuGet, with VS2010.

@jamietre

This comment has been minimized.

Show comment
Hide comment
@jamietre

jamietre Jun 21, 2012

Owner

This is an easy fix. :visible just just checks CSS visibility state, not for non-rendered elements. I'll fix it shortly.

I'm just about to push an update with some major improvements to the HTML parser (auto tag generation that complies with the full HTML5 spec concerning optional opening & closing tags). I'll fix this before I make that push. I probably won't update the release on NuGet for a couple more days since there are so many changes to the parser but feel free to use the dev code at that point. Should be done in an hour or so.

Owner

jamietre commented Jun 21, 2012

This is an easy fix. :visible just just checks CSS visibility state, not for non-rendered elements. I'll fix it shortly.

I'm just about to push an update with some major improvements to the HTML parser (auto tag generation that complies with the full HTML5 spec concerning optional opening & closing tags). I'll fix this before I make that push. I probably won't update the release on NuGet for a couple more days since there are so many changes to the parser but feel free to use the dev code at that point. Should be done in an hour or so.

jamietre added a commit that referenced this issue Jun 21, 2012

@jamietre

This comment has been minimized.

Show comment
Hide comment
@jamietre

jamietre Jun 21, 2012

Owner

Fixed in current push, you can download the compiled dll under the "distribution" folder before I get nuget updated. Let me know if you have any trouble -- the HTML parser has a lot of changes.

Owner

jamietre commented Jun 21, 2012

Fixed in current push, you can download the compiled dll under the "distribution" folder before I get nuget updated. Let me know if you have any trouble -- the HTML parser has a lot of changes.

@jamietre jamietre closed this Jun 21, 2012

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