Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix fallback on subsequent name options not set #11

Merged
merged 1 commit into from
Apr 8, 2024

Conversation

AstrickHarren
Copy link
Contributor

@AstrickHarren AstrickHarren commented Mar 20, 2024

From the CSL Spec it is suggested that

et-al-subsequent-min / et-al-subsequent-use-first
    If used, the values of these attributes replace those of respectively
    et-al-min and et-al-use-first for subsequent cites
    (cites referencing earlier cited items).

Therefore we need to fallback if et_al_subsequent are None

Fixes #10
Also fixes typst/hayagriva#141, and fixes typst/typst#3699

From https://docs.citationstyles.org/en/stable/specification.html?#name
it is suggested that
```
et-al-subsequent-min / et-al-subsequent-use-first
    If used, the values of these attributes replace those of respectively
    et-al-min and et-al-use-first for subsequent cites
    (cites referencing earlier cited items).
```
Therefore we need to fallback if `et_al_subsequent` are `None`

Fixes typst#10
@reknih reknih merged commit 89485c4 into typst:main Apr 8, 2024
2 checks passed
@reknih
Copy link
Member

reknih commented Apr 8, 2024

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants