Skip to content

Add illustrator creator type#16

Open
adunning wants to merge 1 commit into
zotero:masterfrom
adunning:illus
Open

Add illustrator creator type#16
adunning wants to merge 1 commit into
zotero:masterfrom
adunning:illus

Conversation

@adunning
Copy link
Copy Markdown
Contributor

Copilot AI review requested due to automatic review settings January 14, 2026 15:07
@adunning
Copy link
Copy Markdown
Contributor Author

@dstillman Is this the correct way to add a CSL variable not yet represented by a Zotero field? I understand that labels need to be added in a separate location but I am not quite sure where to find them.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds support for the CSL illustrator creator type to enable proper citation of illustrated books in Zotero. The illustrator creator type is added to book and bookSection item types, and mapped to the corresponding CSL field.

Changes:

  • Added illustrator creator type to the book item type
  • Added illustrator creator type to the bookSection item type
  • Added CSL mapping for illustrator to the corresponding CSL field name

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread schema.json
"executiveProducer": "executive-producer",
"guest": "guest",
"host": "host",
"illustrator": "illustrator",
Copy link

Copilot AI Jan 14, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The "illustrator" creator type has been added to the CSL names mapping, but it's missing localized strings in all 48 locale sections. Each locale's "creatorTypes" object needs an "illustrator" entry. For example, in the "af-ZA" locale around line 3712, an entry like "illustrator": "Illustrator" should be added between "host" and "interviewee" (in alphabetical order). This pattern needs to be repeated for all locales including ar, bg-BG, ca-AD, cs-CZ, da-DK, de-AT, de-CH, de-DE, el-GR, en-GB, en-US, es-CL, es-ES, es-MX, et-EE, eu, fa, fi-FI, fr-CA, fr-FR, gl-ES, he-IL, hr-HR, hu-HU, id-ID, is-IS, it-IT, ja-JP, km-KH, ko-KR, lt-LT, lv, nb-NO, nl-NL, nn-NO, pl-PL, pt-BR, pt-PT, ro-RO, ru-RU, sk-SK, sl-SI, sr-RS, sv-SE, ta-IN, th-TH, tr-TR, uk-UA, vi-VN, zh-CN, and zh-TW. Without these locale strings, the illustrator creator type will not display properly in the Zotero user interface for different languages.

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants