PR: Merge Development branch into Master #4
Conversation
index.html
Outdated
|
|
||
| <body> | ||
| <textarea class="col-xs-12 col-sm-6 col-md-6 col-lg-6"> | ||
| <textarea id="inputText" class="col-xs-12 col-sm-6 col-md-6 col-lg-6"> |
There was a problem hiding this comment.
This textarea doesn't need id, it is not used anywhere.
hermanzdosilovic
left a comment
hermanzdosilovic
left a comment
There was a problem hiding this comment.
When I press B button to add bold text, it allows me to continue typing. But when I press some LaTeX button, my cursor is gone and I need to click again between $$ or $$$$.
index.html
Outdated
|
|
||
| <script> | ||
| function insertAfterCursor(string) { | ||
| var cursorCh = simplemde.codemirror.getCursor()["ch"] |
index.html
Outdated
| insertAfterCursor("$$ $$") | ||
| }, | ||
| className: "fa fa-area-chart", | ||
| title: "Latex Equation" |
There was a problem hiding this comment.
Both Latex Equation and Latex Expression are equations, only difference is the way they are shown, one is inline and other is block. So I think we should call these Inline math and Math block.
index.html
Outdated
| "guide" | ||
| ], | ||
| "insertTexts": { | ||
| latexEq: ["", "%%"] |
There was a problem hiding this comment.
I think this is not valid. We don't use % anywhere. https://github.com/NextStepWebs/simplemde-markdown-editor#configuration Check again what this configuration property means
Finished editing SimpleMDE toolbar and ready to merge