Fix Misaligned Annotation Rendering

[habesoftware-claude-connector]

Before

After

Summary

Fixes IIIFDataService.parseXYWH, which previously only handled selectors of the form xywh=pixel:x,y,w,h. Bare xywh=x,y,w,h (the W3C Media Fragments default, and what TPEN-Prompts emits) and URI-fragment forms like https://canvas.uri/#xywh=... produced NaN for x, causing every annotation overlay to render glued to the left edge of the canvas.

Replaces the brittle replace(...).split(",") parse with a regex that tolerates:

• no prefix: xywh=100,200,300,400
• pixel: prefix: xywh=pixel:100,200,300,400
• pct: prefix: xywh=pct:10,20,30,40 (parsed, not yet converted to pixels — see Known Limitations)
• URI fragments: https://canvas/#xywh=...
• negative and decimal coordinates
• null / undefined input (returns the zero box)

On no match, returns { x: 0, y: 0, w: 0, h: 0 } instead of propagating NaN into the DOM.

Closes

• Closes parseXYWH fails when selector omits 'pixel:' prefix — lines render at x=0 #7
• Resolves the Page-Viewer-side root cause of 'View Full Page' Annotation Renderings are Wrong TPEN-interfaces#539

Files Changed

iiif-data-service.js | 13 +++++++++----
1 file changed, 9 insertions(+), 4 deletions(-)

Known Limitations

• pct: selectors are syntactically accepted but the values flow into ui-manager.js as if they were pixels, so a real percentage selector will render at the wrong location. Tracked separately — out of scope for this fix.
• The numeric character class is [\d.]+, which would accept malformed tokens like 1.2.3 (yielding NaN). Considered acceptable for trusted IIIF input; can be tightened in a follow-up.