Skip to content

No dl in HTML#607

Merged
jonludlam merged 13 commits intoocaml:masterfrom
dbuenzli:no-html-dls
Mar 9, 2021
Merged

No dl in HTML#607
jonludlam merged 13 commits intoocaml:masterfrom
dbuenzli:no-html-dls

Conversation

@dbuenzli
Copy link
Copy Markdown
Contributor

This gets rids of dl in HTML.

In order to do so:

  1. We enrich the IR's description lists to be able to have classes on description pairs.
  2. We tweak the IR representation for @-tags along the line mentioned in Improve markup for @-tags #329
  3. We tweak the IR representation for module synopses (no spurious p)
  4. We tweak and cleanup the CSS file a bit.

Btw. it seems the ocp-indent file is not consistent with what ocamlformat wants, e.g. w.r.t. identation. It's quite annoying.

@dbuenzli
Copy link
Copy Markdown
Contributor Author

This is best reviewed per commit.

@Drup
Copy link
Copy Markdown
Contributor

Drup commented Feb 26, 2021

This looks pretty good to me! Do you intend to add the collapse in another PR ? I'm surprised there is no impact on the man/latex tests, but I guess that's good. :)

@dbuenzli
Copy link
Copy Markdown
Contributor Author

Do you intend to add the collapse in another PR ?

If I don't manage to do it by the end of the week-end, I'll let that to someone else.

That is the pair itself, not only its components.
Tags are mapped to description items classified by the tag name. We
distinguish in a tag: the tag name, a value and a description.

The tag name and value are separated by a space, properly classified
and define the item term. The description defines the item definition.
N.B. that `.sel` class seems unused in the code base.
Cleaner than having their structure in Block.
@lubegasimon
Copy link
Copy Markdown
Collaborator

This gets rids of dl in HTML.

Why do you want to do that? The reason I’m asking is that I want to do some reviews, which requires me to know the reason behind your implementation.

@dbuenzli
Copy link
Copy Markdown
Contributor Author

dbuenzli commented Mar 6, 2021

Why do you want to do that?

See the dicussion here #329.

@lubegasimon
Copy link
Copy Markdown
Collaborator

Why do you want to do that?

See the dicussion here #329.

Thanks!

@lubegasimon
Copy link
Copy Markdown
Collaborator

LGTM!

Copy link
Copy Markdown
Contributor

@Drup Drup left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jonludlam the CI failure is spurious, please fire away. :)

@jonludlam
Copy link
Copy Markdown
Member

In it goes!

@jonludlam jonludlam merged commit 4a97cf8 into ocaml:master Mar 9, 2021
@jonludlam
Copy link
Copy Markdown
Member

Thanks, @dbuenzli !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants