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

Ignore single tags, e.g. <br /> #2

Closed
lervag opened this issue Nov 22, 2011 · 3 comments
Closed

Ignore single tags, e.g. <br /> #2

lervag opened this issue Nov 22, 2011 · 3 comments

Comments

@lervag
Copy link

lervag commented Nov 22, 2011

Hi,

I just tested this plugin, and when I tried it on a random XML-file I discovered that it had a minor issue that I think should be relatively easy to fix. Consider the following xml-code:

  <property name="&lt;Super&gt;p" type="empty"/>
  <property name="&lt;Control&gt;Escape" type="empty"/>
</property>

In this case, when the cursor is on the second "property" tag, it should not match the -tag. However, it does, since it does not recognize the closing "/>" as making the tag a "single tag".

@gregsexton
Copy link
Owner

Yes this will definitely cause a problem. I'll take a look at fixing this and update when I have a fix.

Thanks,

Greg

@gregsexton
Copy link
Owner

I've just pushed a series of commits. This should be fixed now. Let me know if you still have any problems.

Thanks,

Greg

@lervag
Copy link
Author

lervag commented Nov 23, 2011

Great! Good work!

  • Karl

2011/11/23 Greg Sexton
reply@reply.github.com:

I've just pushed a series of commits. This should be fixed now. Let me know if you still have any problems.

Thanks,

Greg


Reply to this email directly or view it on GitHub:
#2 (comment)

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

No branches or pull requests

2 participants