The width of documentation of CGAL::Polygon_mesh_processing::corefine_and_compute_boolean_operations is too large

[lrineau]

The width of documentation of CGAL::Polygon_mesh_processing::corefine_and_compute_boolean_operations is too large

[sloriot]

If you use the horizontal scrolling bar it works. It's not broken. I can rename the template parameters if you want.

[lrineau]

It is not broken, but it make the documentation really difficult to read. In particular the paragraph:

The positions of the meshes in the array output are specific to the Boolean operation to compute and Corefinement::Boolean_operation_type encodes and describes the ordering. Constructing the default array means that no Boolean operation will be done. Overwriting a default value will trigger the corresponding operation. In such a case, the address to a valid surface mesh must be provided. The optional named parameters for all output meshes are provided as a tuple and follow the same order as the array output. A call to corefine_and_compute_boolean_operations() with optional named parameters passed for output meshes should be done using make_tuple() as the types of named parameters are unspecified.

On my screen, it is really large, twice wider than the screen:


and it is difficult to follow the stream of the sentences, using the horizontal scrollbar.

[afabri]

Let's just change it to NPOut1

[afabri]

Tested in CGAL-Ic-237
I am wondering if we should not also have a shorter namespace PMP.

[lrineau]

Tested in CGAL-Ic-237 I am wondering if we should not also have a shorter namespace PMP.

I plan to merge #8178. Do you think we should keep this issue open, and add another patch to have a nicer documentation? @afabri