-
Notifications
You must be signed in to change notification settings - Fork 56
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
IGrammar.tokenizeLine not considering TextMate format strings properly #116
Comments
@fabioz you consume the API correctly. I had written quicly that at https://github.com/eclipse/tm4e/wiki/Core In short answern this issue seems to be a bug. But VSCode textmate seems have the same problem. In their VSCode installation, they use a markdown grammar with simple declaration instead of To be honnest with you, I'm not a big expert with TextMate. I have just translated the vscode-textmate written in TypeScript to Java to have the same feature than VSCode (I have almost the same tests). The best thing to fix this issue is to create an issue at https://github.com/Microsoft/vscode-textmate/issues My API is the same than vscode-textmate. Hope you will understand. |
Reported there too: microsoft/vscode-textmate#48 |
The output of the given test is currently:
But it should actually replace
${1/(#)(#)?(#)?(#)?(#)?(#)?/${6:?6:${5:?5:${4:?4:${3:?3:${2:?2:1}}}}}/}
by a 2 in this case (actually, the number of # in the parsed text) -- i.e.:
markup.heading.2.markdown
The TextMate manual says that to support this
In Format Strings
In Both
Variables
Code to reproduce below (grammar gotten from https://github.com/textmate/markdown.tmbundle).
It's possible that I'm just misreading the API and there's some post-processing to do after the match. If this is the case, please let me know.
The text was updated successfully, but these errors were encountered: