Automattic / Automattic/node-canvas

Use `cairo_show_text_glyphs`

Open
#2,611 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
10.7k
Forks
1.2k
Avg merge
4d 8h
Merged PRs (30d)
1

Description

I forgot to do this in #2602

https://github.com/Automattic/node-canvas/blob/8ccddbe553d067936f289db61a6fca87c4b9b3bd/src/CanvasRenderingContext2d.cc#L2546-L2547

This is a harder API to use, but I need to do it so that v4 doesn't break text being embedded in PDFs.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start at src/CanvasRenderingContext2d.cc around the linked lines and compare the existing text-rendering path with Cairo's cairo_show_text_glyphs API. Verify the change against PDF output; done means text remains embedded rather than breaking in v4.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp, javascript
Domain
computer-graphics
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.