Skip to content

Draw connector lines below their points instead of on top #42

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.

Sign up for GitHub

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

Open
cduvallet wants to merge 2 commits into
base: main
Choose a base branch
from
+14 −20
Open

Draw connector lines below their points instead of on top #42

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
bccbbe4
Draw connector lines below their points instead of on top
cduvallet Jul 23, 2026
a5c4692
Simplify line zorder default, drop new param per maintainer feedback
cduvallet Jul 24, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
34 changes: 14 additions & 20 deletions src/textalloc/__init__.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -123,7 +123,7 @@ def allocate(
auto_ha (bool, optional): If True, horizontally aligns text dependent on location relative to it's x and y coordinate. Defaults to True.
xlims (Tuple[float, float], optional): x-axis limits of the plot. Defaults to ax.get_xlim().
ylims (Tuple[float, float], optional): y-axis limits of the plot. Defaults to ax.get_ylim().
plot_kwargs (dict, optional): kwargs for the plt.plot of the lines if draw_lines is True.
plot_kwargs (dict, optional): kwargs for the plt.plot of the lines if draw_lines is True. Can override linewidth, color, and zorder (lines default to a zorder just below the scatter point's, so the point is drawn on top of the line).
**kwargs (): kwargs for the plt.text() call.

Returns:
Expand Down Expand Up @@ -280,6 +280,12 @@ def allocate(
scatter_plot_bbs = get_scatter_bbs(scatter_plot, ax)
scatter_plot_bbs[:, 2] = scatter_plot_bbs[:, 0] + scatter_plot_bbs[:, 2]
scatter_plot_bbs[:, 3] = scatter_plot_bbs[:, 1] + scatter_plot_bbs[:, 3]
# Default the line's zorder below matplotlib's default zorder for scatter/patch
# collections (1), so the point marker is drawn on top of and visually caps the
# line where it meets it. Override via plot_kwargs={"zorder": ...} if needed.
line_zorder = 0.9
if scatter_plot is not None:
line_zorder = min(line_zorder, scatter_plot.get_zorder() - 0.1)

# Process extracted textboxes
if verbose:
Expand Down Expand Up @@ -361,27 +367,15 @@ def allocate(
line_objects.append(None)
else:
x_, y_, z_ = line
line_kwargs = dict(
linewidth=linewidth, c=linecolor[ind], zorder=line_zorder
)
if plot_kwargs is not None:
line_kwargs.update(plot_kwargs)
if z_ is not None:
line_objects.append(
ax.plot(
x_,
y_,
z_,
linewidth=linewidth,
c=linecolor[ind],
**(plot_kwargs if plot_kwargs is not None else {}),
)[0]
)
line_objects.append(ax.plot(x_, y_, z_, **line_kwargs)[0])
else:
line_objects.append(
ax.plot(
x_,
y_,
linewidth=linewidth,
c=linecolor[ind],
**(plot_kwargs if plot_kwargs is not None else {}),
)[0]
)
line_objects.append(ax.plot(x_, y_, **line_kwargs)[0])

# Get texts
result_text_xyz: List[Optional[Tuple[float, float]]] = [None] * len(text_list)
Expand Down