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

Fixing latex input render in math-tex #33

Closed
wants to merge 4 commits into from
Closed

Fixing latex input render in math-tex #33

wants to merge 4 commits into from

Conversation

tdaron
Copy link

@tdaron tdaron commented Mar 10, 2023

Hey !
I tried to fix my problem, here is my solution !

This is probably not a beautiful solution, but it actually works !

(i think there should be a switch in parameters to enable/disable it)

Here is before:
image

q_etu = 5
toto = 2

here is after (using my pr):
image

q_{etu} = 5
toto = 2

@gtg922r
Copy link
Owner

gtg922r commented Mar 11, 2023

Thanks for the PR, I really appreciate it! mathjs uses {...} to define objects. Stripping braces out would prevent objects from being used. Let me do some more research and see if that would be an actual problem, or if there are any other options for ways to enable this.

Its possible with a regex I can only pull out braces from inside a variable name which would limit the damage

@gtg922r
Copy link
Owner

gtg922r commented Mar 11, 2023

I'm trying to close out fixing of the number formatting issue (the bug you originally surfaced), including letting the user select the desired formatting. Then I'll look at this. I agree it'd be nice to be able to have the subscript in rendered variable names!

@tdaron
Copy link
Author

tdaron commented Mar 12, 2023

Oh i didn't know mathjs used brackets for something, i looked too fast their docs !

Anyway, if needed, i would be glad to help !

@tdaron tdaron closed this by deleting the head repository Nov 15, 2023
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