-
Notifications
You must be signed in to change notification settings - Fork 64
Add experimental support to build using Bikeshed #296
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
base: main
Are you sure you want to change the base?
Conversation
|
Although this is far from complete, I'd like to add the experimental mode to html-build to make collaboration on it easier. At TPAC I'd like to demo how to use it (which is a lot easier if this is merged) and where people can help out if they're interested. I'm happy to clean up some things if requested, I wrote this as an experiment and probably left a few weird things in that need explaining or removing. |
|
Happy to rubber stamp this, but let's give @sideshowbarker and @zcorpan a chance to chime in as well. |
Same here. This is epic and it seems like it could go ahead and be landed — for the reasons Philip outlined in https://github.com/whatwg/html-build/pull/296#issuecomment-3464324423 — and any further work could be handled in follow-up PRs |
|
I tried running this which worked. The output is broken (e.g. many cross-references don't work correctly), but that is as expected. |
|
@foolip can you resolve merge conflicts? |
The new --bikeshed flag enables Bikeshed mode. The wattsi2bikeshed.js
script converts the source from Wattsi to Bikeshed syntax.
The hardest part of this is ensuring that cross-references still resolve
to the same thing. Wattsi's <span> and <i> for autolinks are converted
to <a>, but changes to HTML itself will likely be needed for some cases
where the data-x="" (with hyphens) is unambiguous but where Bikeshed's
linking text (with spaces) is ambiguous and would require a for=""
attribute.
What doesn't work:
- Some cross-references as per above.
- Some manual changes to HTML are still needed:
https://github.com/whatwg/html/tree/bikeshed-experiment
- Existing IDs aren't always preserved.
- Simplification of HTML into Bikeshed shorthands like {{Node}}.
- subdfn, which has no equivalent in Bikeshed.
- Variants like the dev edition aren't properly supported.
fa185ea to
ece38f7
Compare
The new
--bikeshedflag enables Bikeshed mode. The wattsi2bikeshed.js script converts the source from Wattsi to Bikeshed syntax.The hardest part of this is ensuring that cross-references still resolve to the same thing. Wattsi's
<span>and<i>for autolinks are converted to<a>, but changes to HTML itself will likely be needed for some cases where thedata-x=""(with hyphens) is unambiguous but where Bikeshed's linking text (with spaces) is ambiguous and would require afor=""attribute.What doesn't work:
https://github.com/whatwg/html/tree/bikeshed-experiment
{{Node}}.subdfn, which has no equivalent in Bikeshed.