Improve text bounding boxes (fix empty Menu, Textbox issues)

[ffreyer]

Description

This is a more low level fix for #5333. The errors in Menu and Textbox, and probably most around round_to_IRect2D as well, happen because text bounding boxes return NaN (or Inf, -Inf) when a string is zero length. And that happens because we prematurely make text positions per character, effectively removing positions for empty strings.

This pr reworks the text position pipeline to remain per-string as long as possible. They are then included in per-string (and plot) bounding boxes, changing them from Inf/NaN bounding boxes to Rect3d(string_pos, Vec3d(0)). With that:

• raw_glyph_boundingboxes, fast_glyph_boundingboxes and glyph_boundingboxes should be unchanged
• fast_string_boundingboxes now return Rect3d(Point3d(NaN), Vec3d(0)) instead of Rect3d(Point3d(Inf), Vec3d(-Inf)) for an empty string (These skip positions)
• string_boundingboxes should now always be finite (if positions are finite). Before empty strings resulted in Rect3d(Point3d(NaN), Vec3d(0))
• full_boundingbox and boundingboxes now consider empty strings to, i.e. include their position. All empty strings no longer results in Rect3d(Point3d(Inf, Inf, 0), Vec3d(-Inf, -Inf, 0))

Most of these are just plainly improvements imo. The only exception is the last case, where one might argue that an empty string should not be contributing to the bounding box of the plot, since it results in no visualization.

• fixes Empty strings no longer supported for Menu block options. #5333
• fixes text from SliderGrid observables errors out #5392
• fixes Legend with empty labels crashes in CairoMakie 0.15 but works in 0.14 #5311
• fixes Label initialized with empty string does not show new text #5365

Type of change

Maybe slightly breaking because text.text_positions got deleted, and various other positions in text are now per-string instead of per-char. Though I'd consider those changes internal.

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

• Added an entry in CHANGELOG.md (for new features and breaking changes)
• Added or changed relevant sections in the documentation
• Added unit tests for new algorithms, conversion methods, etc.
• Added reference image tests for new plotting functions, recipes, visual options, etc.

[MakieBot]

Benchmark Results

SHA: 36ce0633b0cb8c4edd6b04edce49c13c2598e446

Warning

These results are subject to substantial noise because GitHub's CI runs on shared machines that are not ideally suited for benchmarking.

[ffreyer]

This fails colorbuffer(ax) because some text data_limits have become finite. Probably xlabel, ylabel, title: (before vs after, see top left and bottom right. The black dot is CairoMakie artifact)

Makie.jl/Makie/src/makielayout/blocks/axis.jl

Lines 1985 to 1989 in c9c5882

	function axis_bounds_with_decoration(axis::Axis)
	# Filter out the zoomrect + background plot
	lims = Makie.data_limits(axis.blockscene.plots, p -> p isa Mesh || p isa Poly)
	return Makie.parent_transform(axis.blockscene) * lims
	end

@jkrumbiegel Can't we calculate the pixel boundingbox of an Axis with decorations more directly from layoutobservables? Computedbbox with protrusions maybe?

[ffreyer]

bbox + protrusions doesn't consider the font size so sometimes ticks can get cut off