SemianiakaVY/openpencil
1
0
Fork 0
Code Issues Pull requests Actions 1 Packages Projects Releases Wiki Activity

fix(clipboard): match shaped glyphs by character index, not array position

Browse source
This commit is contained in:
Danila Poyarkov 2026-05-16 18:38:58 +03:00
parent 578b93e189
commit ea323c33f0
1 changed files with 6 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
packages/core/src/text/derived-text/clipboard.ts
View file

@ -103,6 +103,11 @@ export async function buildDerivedTextDataV4(
: computeWordWrapBreaks(node.text, glyphMetrics, fallbackAdvance, node.width, node.fontSize, node.fontFamily) : computeWordWrapBreaks(node.text, glyphMetrics, fallbackAdvance, node.width, node.fontSize, node.fontFamily)
const lineBreakSet = new Set(lineBreaks) const lineBreakSet = new Set(lineBreaks)
const shapedByChar = new Map<number, (typeof shaped extends null | undefined ? never : NonNullable<typeof shaped>)['glyphs'][number]>()
if (shaped) {
for (const g of shaped.glyphs) shapedByChar.set(g.firstCharacter, g)
}
const fallbackBaselines: NonNullable<NodeChange['derivedTextData']>['baselines'] = [] const fallbackBaselines: NonNullable<NodeChange['derivedTextData']>['baselines'] = []
const fallbackOffsets = Array.from({ length: node.text.length + 1 }, () => 0) const fallbackOffsets = Array.from({ length: node.text.length + 1 }, () => 0)
let fallbackX = 0 let fallbackX = 0
@ -110,7 +115,7 @@ export async function buildDerivedTextDataV4(
let lineStart = 0 let lineStart = 0
const glyphs = glyphMetrics.map((glyph, index) => { const glyphs = glyphMetrics.map((glyph, index) => {
const shapedGlyph = shaped?.glyphs[index] const shapedGlyph = shapedByChar.get(index)
const fallbackGlyphAdvance = glyph.advance || fallbackAdvance const fallbackGlyphAdvance = glyph.advance || fallbackAdvance
if (!shapedGlyph && lineBreakSet.has(index)) { if (!shapedGlyph && lineBreakSet.has(index)) {
fallbackBaselines.push({ fallbackBaselines.push({