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
Consolidate styles and scripts in Bikeshed-generated standards #68
Comments
cc @tabatkins |
Bikeshed inlines those styles so that random specs don't magically gain a dependency on some other org's domain. If you want do do it yourself, tho, that's easy - tell Bikeshed to omit the styles with |
Could you explain what you mean by the first sentence? (Also, thanks for the advice, that'll help a ton.) |
Bikeshed's syntax highlighting, for example, depends on some CSS to work right. If that CSS was hosted somewhere, anywhere, it would make all Bikeshed documents using syntax highlighting depend on that hosting domain. This isn't great. Instead, Bikeshed inlines the styles so the spec is fully functional all by itself. The only external dependencies a Bikeshed document can have are those imposed explicitly by the group's boilerplate. |
@sideshowbarker it seems the main missing thing here is MDN annotations. Are we already tracking that somewhere? |
Well the MDN-annotations patch for Bikeshed hasn’t landed yet— But as far as I understand, what we’ll end up doing is what’s discussed at speced/bikeshed#1564 (comment) |
Okay, so it should be all good in the end. dom.spec.whatwg.org already has MDN annotations as far as I can tell which is why I was asking. |
Just the review draft, because I built it with Bikeshed from the speced/bikeshed#1564 branch |
whoops, sorry i dropped the ball there. I'll review and merge the pr tomorrow. |
Oh wait, I confused MDN and dfn. My bad. The definitions panel is what is causing inline styles and scripts. |
OK yeah I think there’s no reason we can’t move the dfn stuff to the external stylesheet and script |
I also noticed that whatwg/whatwg.org@4a772bd#diff-1ecbbc7389dd3ea567573f6e14a5fc8c has features that bikeshed's script does not have, such as I have continued to markup things as |
If you'd like any specific feature added to Bikeshed's functionality, just file an issue. ^_^ |
Per IRC discussion Bikeshed has the support we need. We'd have to add omit script-dfn-panel and omit style-dfn-panel to our boilerplate. Remaining things:
|
standard.css
I don't understand this question
I think we should be able to align the styles without much trouble at all. It seems like just changing some class names, plus figuring out if there's any of the HTML-specific style rules we want to port to Bikeshed specs too.
Seems like a script problem?
I don't understand this question. |
@domenic Bikeshed currently inserts an inline script. I'd like us to not do that and host our own script. I was also wondering if the service worker script should be external as well as per OP. |
The service worker script is already external? Or do you mean the one-liner that installs the service worker? |
Currently there's inline style for syntax highlighting and dfn boxes. And also inline script for service workers and dfn boxes. It seems we should consolidate these somewhat as it's not exactly best practice.
Follow-up to #23.
The text was updated successfully, but these errors were encountered: