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

Handle more elements #7

Draft
wants to merge 11 commits into
base: main
Choose a base branch
from
Draft

Handle more elements #7

wants to merge 11 commits into from

Conversation

emdash-ie
Copy link
Collaborator

The converting server currently ignores many elements! For the purpose of demoing, it’d be nice for it to support some more.

I think these should both map to what composer enters for shift-enter,
i.e. a `<br>` tag.
I may have got the title and link text backwards, but I don’t think so:
_title here has type Text while linkText has type Inlines, which sounds
like it matches the description in the manual:

> The link text can contain formatting (such as emphasis), but the title
> cannot.
I _think_ this should work…
This is somewhat hackish, but hopefully it works ok!
At the moment things don’t look quite right: markdown paragraphs aren’t
getting wrapped into single lines. I think this is why, though I might
need to also change the handling of LineBreak in the same way.
The content type used for submission needs to be
application/octet-stream, for servant to accept it. (Otherwise it
returns a 415.)
For converting docx, the browser sends cors preflight requests, which
fail. This middleware should allow them to be answered correctly.
Now that I have automatic headers from a dependency, mine are unneeded!
And causing problems with firefox, so let’s remove them.
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.

1 participant