FingerText icon indicating copy to clipboard operation
FingerText copied to clipboard

Multiple hotspots triggered w/o warning using angle bracket arguments

Open zyrolasting opened this issue 12 years ago • 5 comments

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?

zyrolasting avatar Apr 28 '12 17:04 zyrolasting

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.

erinata avatar Apr 28 '12 18:04 erinata

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

zyrolasting avatar Apr 28 '12 18:04 zyrolasting

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

erinata avatar Apr 28 '12 18:04 erinata

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??

erinata avatar Apr 29 '12 20:04 erinata

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!!

zyrolasting avatar Apr 29 '12 20:04 zyrolasting