-
Notifications
You must be signed in to change notification settings - Fork 7
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
Comma cannot be used or escaped for keyword/trigger #19
Comments
Thank you for the bug report. This is because I use these separators
to break a sentence into words. For ex:-
into an array like
I then use the array to look in the dictionary for keywords. So, it's not great for triggers starting with those separators. I don't remember why I added the ',' into the separator list, may have been added to fix some bug before 😅. I guess I will remove it next update and see if no bugs pop up. Of course, this is assuming I understood what you wanted exactly. I assumed you don't want
but instead want
EDIT: Just realized the solution is to add ", " into the separator list (with a space character) instead of ",". I'll include it in the next update. |
Probably some parsing thing, to make generating the json array easier on import/export? Considering that commas are fine in the expanded text, but not in keywords.
Correct, I use Ideally it should work with something like this:
Or any arbitrary amount/placement of commas. |
v6.9.1. Sorry this took way too long because I was trying to include bigger features in this release (forms support) but they need more testing. |
Trying to use a comma in the keyword does not work at all, however it can be used in the replacement text.
Suppose we have the following espanso yml:
With this, the
,,t
would be interpreted literally. However in TT, this comes out as something like:(note the double quotes)
I might be missing something, but I have not found a way to interpret a keyword as a string literal. Importing a yml file (as mentioned above) does not work, most likely due to how it's converted into a json-friendly format.
Weirdly, other problematic characters are fine (
"
,\
,/
, etc).System info:
The text was updated successfully, but these errors were encountered: