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

Already on GitHub? Sign in to your account

Multiple hotspots triggered w/o warning using angle bracket arguments #29

Open
zyrolasting opened this Issue Apr 28, 2012 · 5 comments

Comments

Projects
None yet
2 participants

Using 0.5.52.0 Alpha

This is the markup I'm using:

<nav>
    <a href="/">Link</a>
    <a href="/">Link</a>
    <a href="/">Link</a>
    <a href="/">Link</a>
</nav>
<article>
    $[![]!]
</article>
<aside>
    $[![]!]
</aside>
<footer>
    $[![]!]
</footer>

I have the following snippet for "link": <link />[>END<]

When I hit TAB by the closing '>' after an </a> containing the word "Link", FingerText does this:

<nav>
    <a href="/">Link</a>
    <a href="/">Link</a>
    <a href="/">Link</a>
    <a href="/"><link />
</nav>
<article>
    /a
</article>
<aside>
    /a
</aside>
<footer>
    /a
</footer>

I figured /a was considered an argument, since it does the same with Link(/a). I found this by accident, but I assume it was by design.

Thing is, I have a bunch of HTML snips and many of them use tag names as snippet names. Considering FT is not case-sensitive, I am now concerned over how many unwanted invocations I'll trigger. Can I at least not use <> to wrap parameters?

Owner

erinata commented Apr 28, 2012

Thanks a lot for reporting this.

use < > to wrap parameter is not by design. It's just because notepad++ considers < > as parenthesis too and I forget to rule it out. I assume that people should be using ( ) to wrap parameters. let me fix it later.

Thanks so much! Unless you need more info from me, I have nothing more
to add.

Owner

erinata commented Apr 28, 2012

Well........in your opinion, is it better (in general, not only about the issue in this thread) if fingertext triggertext is case sensitive?

Owner

erinata commented Apr 29, 2012

http://dl.dropbox.com/u/7429931/fingertext/FingerText%20-%200.5.59.1.zip

Can you try this version 0.5.59.1 and see if that fixed that issue of parameters warpping??

Yes, the problem in the OP is only replicated when using parenthesis. In
the case of my example markup, angle brackets no longer cause the issue.

EDIT: As for case sensitivity, I would include it to prevent unwanted triggers where uppercase letters are conventional, such as VBA or SQL. I would go for it.

Thank you!!

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