-
Notifications
You must be signed in to change notification settings - Fork 12.9k
Images in Language Specification #6144
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
Conversation
@@ -47,6 +47,7 @@ function convertDocumentToMarkdown(doc) { | |||
var tableColumnCount; | |||
var tableCellIndex; | |||
var columnAlignment = []; | |||
var imageCount = 0; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't see this in the original .ts
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I removed it again.
Can we remove the generated Also, where exactly are you saving the images themselves? It's not clear from the code here. |
agree with @DanielRosenwasser about removing generated javascript file. |
var level = 1; | ||
var sectionBreak = text.indexOf("\x0C") >= 0; | ||
|
||
text = trimEndFormattingMarks(text); | ||
if (text === "/") { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What exactly does this mean? Can you leave a comment with the explanation?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure, I will add a comment. An inline image shows up in the text just as a "/". When I see a paragraph consisting of nothing but "/", I check to see if the paragraph contains hidden text and if so I emit that instead. The hidden text is assumed to contain an appropriate markdown image link. I could look at automatically extracting the images, but I'm not sure how much time I want to sink into further exploring the Word object model.
I get that, I'm more curious as to what in the code exactly does that. Was extracting images out a manual process, or is it automated now? As long as we're generally confident we won't be adding more images, either is fine. |
👍 |
Images in Language Specification
Fixes #1566.