ocamldoc should really stop generating br tags

[vicuna]

Original bug ID: 7351
Reporter: @dbuenzli
Assigned to: @Octachron
Status: resolved (set by @Octachron on 2017-03-15T22:06:19Z)
Resolution: fixed
Priority: normal
Severity: minor
Version: 4.03.0
Target version: 4.05.0 +dev/beta1/beta2/beta3/rc1
Fixed in version: 4.06.0 +dev/beta1/beta2/rc1
Category: ocamldoc
Monitored by: @gasche @dbuenzli

Bug description

They prevent using CSS to its full extent to style the documentation. For example here's what I have in my style sheet.

br { display: none } /* Annoying, hide them. /
code br { display: block } / Except in signatures. */

The presence of br in the output disallows me to use e + e' CSS rules. A particular instance of this in my stylesheet is:

hr
{ display: none } /* Would be nice to have but we cannot get that to
interact well with our h1's because of br markup noise */

I'd like to specify something for hr + h1 but I'm unable since the output has the sequence hr br h1

[vicuna]

Comment author: @Octachron

I agree that most use of "br" in ocamldoc html backend are not particularly meaningful and their intend could be better expressed by more semantic tags.
I have proposed a pull request in this direction in #802.

[vicuna]

Comment author: @Octachron

The above patch has been merged in Ocaml developpement branch, and should be integrated in OCaml 4.06.