You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
hi! I'm trying this out after hop and really like it so far, but I've noticed and been able to reproduce a couple of issues with the matcher. Both on this sample text:
hi link @definition Normalhi link @definition.constantConstanthi link @definition.functionFunctionhi link @definition.method Functionhi link @definition.varNormalhi link @definition.parameter Identifierhi link @definition.macroMacrohi link @definition.typeTypehi link @definition.field Todohi link @definition.enumStructurehi link @definition.namespace Definehi link @definition.importIncludehi link @definition.associated Todo
(extra lines on purpose)
with cursor on an empty line, pounce and search dfnt. All instances of definition match, but the "best" match is on line 8 instead of nearest to the cursor. The line 8 match also ignores the first n and extends out to the t in type; there are other cases in which this could happen (e.g. definition.import), but it doesn't.
with cursor on the last line with text, pounce and search dfnt. Only lines 8 and 13 remain candidates by the last character even though all instances of definition matched above. The extended line 8 match is also evident again.
The text was updated successfully, but these errors were encountered:
hi! I'm trying this out after hop and really like it so far, but I've noticed and been able to reproduce a couple of issues with the matcher. Both on this sample text:
(extra lines on purpose)
dfnt
. All instances ofdefinition
match, but the "best" match is on line 8 instead of nearest to the cursor. The line 8 match also ignores the firstn
and extends out to thet
intype
; there are other cases in which this could happen (e.g.definition.import
), but it doesn't.dfnt
. Only lines 8 and 13 remain candidates by the last character even though all instances ofdefinition
matched above. The extended line 8 match is also evident again.The text was updated successfully, but these errors were encountered: