Skip to content
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

[LaTeX] Added support for inline math ($...$) in a single line #62

Merged
merged 1 commit into from
Mar 29, 2016

Conversation

r-stein
Copy link
Collaborator

@r-stein r-stein commented Mar 29, 2016

This has been requested in #61 and adds support for latex inline math ($...$) in as single line:

inline_math

Multi line support would be way more complicated, since we cannot easily see if we are inside a math block.

@r-stein r-stein merged commit 3b3d29b into aronwoost:master Mar 29, 2016
@r-stein r-stein deleted the latex_inline_math branch March 29, 2016 11:03
@excetara2
Copy link

Just letting you know this does not work as I would expect it to for many examples I have tried. If you put the cursor in front of the 5 it will just highlight 5$ and not highlight the whole $...$. Also another command you can try is $550\Hz$ or $220\alpha$. Both of these if the cursor is on either the number side or command side it only selects the left or right part (eg. $550) and won't select the whole $....$.

@r-stein
Copy link
Collaborator Author

r-stein commented Apr 7, 2016

@excetara2 Yes currently the $ is used as a word character because it is used for variable names in some programming languages. Hence we should use different word character. I have already prepared something for this, but I have still minor struggles with the resulting behavior. However I will update it soon, since it's better than the current one anyway =)

@excetara2
Copy link

@r-stein Yeah I've given it a try and seems a lot better. At least everywhere I tried it seemed to eventually select inside the '$' albeit a bit slowly depending where the cursor was but definitely much better than the previous implementation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants