Serve bootstrap and monaco-editor static files locally #27

Closed
opened 2025-09-12 18:53:02 +02:00 by tapiron · 9 comments
Contributor

Would be great to not rely on cloudflare CDN to use this app.

Would be great to not rely on cloudflare CDN to use this app.
Owner

I thought about it and I decided to let the browser download those assets in the first release to keep it simple.

I'm okay with your suggestion. But beware of not bloating the server by adding multiple MB to the binary. We should find a way to download a specific version (configurable) of the assets currently hosted on Cloudflare, to include them compressed in the binary (or serve from the filesystem if files exist), then either serve them compressed if possible, or decompress before serving them statically.

I thought about it and I decided to let the browser download those assets in the first release to keep it simple. I'm okay with your suggestion. But beware of not bloating the server by adding multiple MB to the binary. We should find a way to download a specific version (configurable) of the assets currently hosted on Cloudflare, to include them compressed in the binary (or serve from the filesystem if files exist), then either serve them compressed if possible, or decompress before serving them statically.
jriou self-assigned this 2025-09-15 08:59:18 +02:00
Owner

@tapiron, I've tried to use npm to download bootstrap and serve those assets locally in #31. Could you tell me if that would work for you please?

I tried to do the same with Monaco but it's slightly more complicated.

@tapiron, I've tried to use npm to download bootstrap and serve those assets locally in #31. Could you tell me if that would work for you please? I tried to do the same with Monaco but it's slightly more complicated.
Author
Contributor

Nice, this is more or less what I had in mind when opening this issue!

I'll test this today or tomorrow on my side and let you know if I have some issues.

Could be nice to add a Makefile target for building npm dependencies, but it can be added later.

Nice, this is more or less what I had in mind when opening this issue! I'll test this today or tomorrow on my side and let you know if I have some issues. Could be nice to add a Makefile target for building npm dependencies, but it can be added later.
Owner

I have added make dependencies in #31.

I have added `make dependencies` in #31.
Author
Contributor

Great, it works well on my side

Great, it works well on my side
Owner

Thank you. The bootstrap part is now included on the main branch in ff92e30232.

Thank you. The bootstrap part is now included on the main branch in ff92e302324a5e3bccb3d626bf8892c559cf2dff.
Owner

It seems that Ace is easier to integrate. Are you open to replace the Monaco Editor with Ace?

It seems that [Ace](https://ace.c9.io/) is easier to integrate. Are you open to replace the Monaco Editor with Ace?
Owner

My pull request to replace Monaco by Ace and to serve assets from a local directory is available on #32.
@tapiron, could you check this out please?

My pull request to replace Monaco by Ace and to serve assets from a local directory is available on #32. @tapiron, could you check this out please?
Owner

Monaco editor replaced by Ace in b5701b5a4d.

Monaco editor replaced by Ace in b5701b5a4d473ff5007ad1f89e0cf25b5923be71.
jriou closed this issue 2025-09-27 09:59:17 +02:00
Sign in to join this conversation.
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: jriou/coller#27
No description provided.