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

Migrarate to typst.ts #28

Closed
kwinso opened this issue Sep 18, 2023 · 1 comment
Closed

Migrarate to typst.ts #28

kwinso opened this issue Sep 18, 2023 · 1 comment

Comments

@kwinso
Copy link

kwinso commented Sep 18, 2023

What about typst.ts? I think it will remove a lot of complexity. Also, the bundle size is smaller than the current implementation.

It can compile in memory file, as shown here.

It will eliminate a lot of troubles like mobile file reading, as we'll stay in JS async land, so there's no need on round trips between rust and js.

I think this will help with focusing on the actual functionality rather than fighting with bugs.

@fenjalien
Copy link
Owner

I've had many suggestions to use typst.ts and in short: no, I do not want to use it.

  • Completely lacking in documentation (I understand this is being worked on)
  • It won't remove any complexity it'll just out source them instead, which would cause further problems
  • No we won't stay in JS async land, typst.ts uses an access model concept which is essentially a bunch of callbacks. These are all required to be synchronous. The only reason it works is because of a synchronous xhr.

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

No branches or pull requests

2 participants