denoland / dotland

[Archived] deno.land website
https://deno.land
MIT License
957 stars 626 forks source link

Reduce `gfm.css` bundle size #2035

Closed kidonng closed 2 years ago

kidonng commented 2 years ago

See https://github.com/denoland/dotland/issues/2021#issuecomment-1054403007 The bundle is generated from upstreamed PR: https://github.com/lucacasonato/deno-gfm/pull/2

image

Look mum no lightweight CSS!

kidonng commented 2 years ago

@crowlKats would you like to review this? The huge bundle is causing too much unnecessary traffic.

crowlKats commented 2 years ago

I wonder, should we maybe rather serve the css from gfm directly instead of having a copy ourselves?

kidonng commented 2 years ago

Probably, I'm just following existing conventions. It's just more convenient to have a static CSS file.