Comments (6)
I don't see a reason why a code change would not be accepted especially since it greatly simplifies our fragile workflow.
I was referring to what I initially suggested: removing CSS from this package completely, which could be deemed too aggressive and breaking for little good.
from deno-gfm.
I'm -1 on not shipping any CSS at all. People who care about the difference between inlining styles and linking a stylesheet already do what makes sense for them (ref: Pyro). People who want to use different CSS can if they want to (we should have better docs on how to do that). I definitely think that our current implementation of bundling CSS is suboptimal and could be improved.
and dedicate styling to people who are doing it seriously in a standalone object.
Are you referring to github-markdown-css
or another project here? Is there more than one project doing this?
from deno-gfm.
People who care about the difference between inlining styles and linking a stylesheet already do what makes sense for them
When the docs say nothing about it, people would not care. Find a real example where:
CSS
is served as an actual file<link rel="stylesheet">
is used to request that file
I definitely think that our current implementation of bundling CSS is suboptimal and could be improved.
Then do not do it at all. This project is initially just for simple markdown rendering for deno.land
, the choice to host styles in house only made sense at that time.
Are you referring to
github-markdown-css
or another project here? Is there more than one project doing this?
As far as I know github-markdown-css
is the only one doing similar things. I said people because, well, it's not just one person working on that.
from deno-gfm.
Find a real example where...
Pyro does that, check out the head of the website.
Then do not do it at all.
This seems dubious to me. I call it suboptimal because I rewrote the old implementation to what we have today and know that the build system is pretty fragile. In my eyes, we would ideally switch to github-markdown-css
instead of trying to do the generation ourselves. I think we should write docs (in general) and tell people who care that they can use a linked stylesheet.
This project is initially just for simple markdown rendering for deno.land, the choice to host styles in house only made sense at that time.
I think it still has some merit in its simplicity today. That's the big selling point of deno-gfm
to me.
I said people because, well, it's not just one person working on that.
Thanks for clarifying, 👍
from deno-gfm.
Pyro does that
I see where you are coming from, but many people are not seasoned enough to make the same choice.
In my eyes, we would ideally switch to
github-markdown-css
instead of trying to do the generation ourselves.
I probably should have clarified, that is the real thing I want to get rid of. I'm not saying the CSS
export should be nuked entirely (which would very unlikely to be accepted by the maintainers), it could be changed to re-export github-markdown-css
.
I think we should write docs (in general) and tell people who care that they can use a linked stylesheet.
I would certainly resort to better docs if changing actual code is not approved.
I think it still has some merit in its simplicity today. That's the big selling point of
deno-gfm
to me.
Same for me. I was just stating why does this package bundles styles to begin with.
from deno-gfm.
I probably should have clarified, that is the real thing I want to get rid of. I'm not saying the CSS export should be nuked entirely (which would very unlikely to be accepted by the maintainers), it could be changed to re-export github-markdown-css.
Ah sorry, I misinterpreted your message. I completely agree with this sentiment. I don't see a reason why a code change would not be accepted especially since it greatly simplifies our fragile workflow.
from deno-gfm.
Related Issues (20)
- Slow load times HOT 5
- Needs CI
- Write more tests HOT 5
- mathify should not be called if opts.allowMath is false HOT 5
- Switch to a CommonMark compliant parser HOT 15
- Replace Prism with starry-night HOT 12
- What is the current scope of this project? HOT 8
- Write actual docs
- katex types seemingly not cacheable HOT 2
- yaml parsing does not work
- title="null" with disableHtmlSanitization: true
- `render`(`mathify`) throws `ParseError` while rendering README.md of deno_mongo
- Rendering bug with dollar signs in tables HOT 1
- /x does not render text as bold if followed by a hard line-break HOT 5
- HTML appears in code HOT 2
- Dependency Deprecation Warning HOT 1
- Support "Note" and "Warning" highlight like GitHub HOT 2
- Remove marked-mangle as it doesn't work HOT 1
- Dependency deprecation warning for `baseUrl` HOT 2
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from deno-gfm.