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
HTML backend: refine specification rendering #615
Conversation
While doing this PR I realized the rendering by the default stylesheet of variants with a doc string is not very nice, I just want to stress that this is not a side effect of this PR. I don't think it's worth fixing before #614 is. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm generally in favor of these changes, except for the copy-paste one. Your solution is, imho, not acceptable, but I'm pretty sure we can do better. The rest is good to go (as soon as the previous PRs are merged).
ae78979
to
db135b3
Compare
We simply use the `attr` pattern already present in the IR model. This also slightly tweaks some of the "kinds": * `extension` becomes `type extension` * `type-subst` becomes `type subst` * `external` becomes `value external` * `instance-variable` becomes `value instance-variable` Fixes part of ocaml#601.
db135b3
to
b716452
Compare
This has been removed.
I suggest we do that in a separate PR once #630 has been resolved. The PR has been rebased |
Thanks! |
This PR is built on top of #612, the relevant commits start at e7d6fd3
It does the following:
->
so that browsers can be instructednot to break it.
type
#601 adds several kinds forItem
. We reuse theClass.t
type usually namedattr
(sic). A few items get reclassified, mostly in a backward compatible way. See the commit message for details..doc
to.def-doc
.odoc.css
with the changes.Tweakodoc.css
for better c&p behaviour (closes Record field and variant case rendering breaks cut and paste #296).