Something that occurred to me when doing the work in #176 - we now have PlaceholderCanvas and AccompanyingCanvas and they can make use of all(?) the helpers that are applicable to Canvas objects and so we should look at updating existing helpers to monkeypatch them in here when appropriate.
Additionally, we might consider extending make_canvas_from_iiif to be able to create either of the other types.
By passing object type in the constructor - slight clunky, breaking change
By copying the existing code into a new helper and changing the object type created - less work, but violates DRY
By wrapping the existing function and type-coercing - unsure if this would work, but it's my favoured approach if it does
Something that occurred to me when doing the work in #176 - we now have
PlaceholderCanvasandAccompanyingCanvasand they can make use of all(?) the helpers that are applicable toCanvasobjects and so we should look at updating existing helpers to monkeypatch them in here when appropriate.Additionally, we might consider extending
make_canvas_from_iiifto be able to create either of the other types.