Skip to content

IIIFの題名の既定をManifestのlabelに統一 - #165

Merged
kochizufan merged 3 commits into
masterfrom
dec26/iiif-title-from-manifest
Nov 10, 2026
Merged

kochizufan merged 3 commits into
masterfrom
dec26/iiif-title-from-manifest

Conversation

@kochizufan

Copy link
Copy Markdown
Member

変更

IIIF の書誌を地図へ写す際、proposeIiifBindings の title の既定を常に { from: "manifest", property: "label" } にしました。canvas の label が「page 1」でも Manifest の書名を提案します。Manifest の label が読めない場合は title を返しません。

利用者が「このコマ」を選ぶための明示的な canvas の指定は維持しています。不要な数字・空文字判定を削除し、README(英日)と CHANGELOG を更新しました。

検証

  • 回帰試験: 修正前に4件失敗、修正後に対象62件成功
  • 全体単体試験: 56ファイル、1002件成功、1件スキップ
  • 型検査: 成功
  • node_modules/.bin/eslint src spec: エラー0件、既存の警告2件
  • pnpm run build: 成功、バンドル検査も成功
  • 明示的な canvas label の解決を確認する既存試験も成功

#145〜#147 に合わせ、build で生成した dist の差分(型定義・ソースマップ)を専用コミットに含めています。マージは行っていません。

🤖 Generated with Claude Code

kochizufan and others added 3 commits November 10, 2026 19:42
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@kochizufan
kochizufan marked this pull request as ready for review November 10, 2026 10:52
@kochizufan
kochizufan merged commit 8ed4d85 into master Nov 10, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant