Skip to content

Clarify fonttype switch in backend_pdf. #31456

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

Merged
QuLogic merged 1 commit into from
Apr 7, 2026
Merged

Clarify fonttype switch in backend_pdf. #31456

Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
38 changes: 13 additions & 25 deletions lib/matplotlib/backends/backend_pdf.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2288,13 +2288,8 @@ def _encode_glyphs(self, subset, fonttype):
return b''.join(glyph.to_bytes(2, 'big') for glyph in subset)

def encode_string(self, s, fonttype):
match fonttype:
case 1:
return s.encode('cp1252', 'replace')
case 3:
return s.encode('latin-1', 'replace')
case _:
return s.encode('utf-16be', 'replace')
encoding = {1: 'cp1252', 3: 'latin-1', 42: 'utf-16be'}[fonttype]
return s.encode(encoding, 'replace')

def draw_text(self, gc, x, y, s, prop, angle, ismath=False, mtext=None):
# docstring inherited
Expand All @@ -2312,29 +2307,13 @@ def draw_text(self, gc, x, y, s, prop, angle, ismath=False, mtext=None):
else:
features = language = None

# For Type-1 fonts, emit the whole string at once without manual kerning.
if mpl.rcParams['pdf.use14corefonts']:
font = self._get_font_afm(prop)
fonttype = 1
else:
font = self._get_font_ttf(prop)
fonttype = mpl.rcParams['pdf.fonttype']

if gc.get_url() is not None:
font.set_text(s, features=features, language=language)
width, height = font.get_width_height()
self.file._annotations[-1][1].append(_get_link_annotation(
gc, x, y, width / 64, height / 64, angle))

# If fonttype is neither 3 nor 42, emit the whole string at once
# without manual kerning.
if fonttype not in [3, 42]:
if not mpl.rcParams['pdf.use14corefonts']:
self.file._character_tracker.track(font, s,
features=features, language=language)
self.file.output(Op.begin_text,
self.file.fontName(prop), fontsize, Op.selectfont)
self._setup_textpos(x, y, angle)
self.file.output(self.encode_string(s, fonttype),
self.file.output(self.encode_string(s, fonttype=1),
Op.show, Op.end_text)

# A sequence of characters is broken into multiple chunks. The chunking
Expand All @@ -2350,6 +2329,9 @@ def draw_text(self, gc, x, y, s, prop, angle, ismath=False, mtext=None):
# Each chunk is emitted with the regular text show command (TJ) with appropriate
# kerning between chunks.
else:
font = self._get_font_ttf(prop)
fonttype = mpl.rcParams['pdf.fonttype']

def output_singlebyte_chunk(kerns_or_chars):
if not kerns_or_chars:
return
Expand Down Expand Up @@ -2400,6 +2382,12 @@ def output_singlebyte_chunk(kerns_or_chars):
self.file.output(Op.end_text)
self.file.output(Op.grestore)

if gc.get_url() is not None:
font.set_text(s, features=features, language=language)
width, height = font.get_width_height()
self.file._annotations[-1][1].append(_get_link_annotation(
gc, x, y, width / 64, height / 64, angle))

def new_gc(self):
# docstring inherited
return GraphicsContextPdf(self.file)
Expand Down
Loading