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

Expose node name #12

Open
wants to merge 1 commit into
base: v2
Choose a base branch
from
Open

Expose node name #12

wants to merge 1 commit into from

Conversation

vtsurkan
Copy link

No description provided.

@niemeyer
Copy link
Contributor

Thanks. Happy to integrate something along these lines, but it seems that this would be better supported by offering the xpath equivalent functionaly: name() and namespace-uri(). With these all the rich expressiveness of the language is also available for the text in these locations.

Also, we need tests for the new functionality.

Are you interested in pursuing those?

@vtsurkan
Copy link
Author

Gustavo
Thanks for your reply. Exposing xpath functionality as you had mentioned would be a awesome, though it doesn't negate the need for the name of the Node to be exposed. As everyone else I am under great stress for deliverables at work, but will try to do something along the lines you suggested.

ThanksVT

 On Wednesday, May 20, 2015 8:54 PM, Gustavo Niemeyer <[email protected]> wrote:

Thanks. Happy to integrate something along these lines, but it seems that this would be better supported by offering the xpath equivalent functionaly: name() and namespace-uri(). With these all the rich expressiveness of the language is also available for the text in these locations.Also, we need tests for the new functionality.Are you interested in pursuing those? —
Reply to this email directly or view it on GitHub.

Copy link

@felinir felinir left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This works. I use HTML tag names to iterate over a <dl>,so I implemented the exact same thing.

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

Successfully merging this pull request may close these issues.

3 participants