-
-
Notifications
You must be signed in to change notification settings - Fork 85
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
year-suffix-delimiter is not actually used #215
Comments
You you give a specific example of items showing this problem, a minimal style with the faulty combination of options, the current output, and the expected output? |
This is the reference set I've been using to test this:
A CSL code is [dashes for exmphasis]
With this code, I'd expect (Neave, 1939a--b--1940a--b). What I actually get is (Neave, 1939a, b, 1940a, b). With
then I get (Neave, 1939a--b--1940a--b). Which is not supposed to happen at least if I read the spec correctly? The spec clearly implies that And finally with I should get (Neave, 1939a; b; 1940a; b) according to the spec's own explicit example because the semicolon is set in "layout". What I actually get is (Neave, 1939a, b, 1940a, b), i.e. the default for As a side note, I cannot tell from reading the spec whether you're supposed to be able to specify a distinct delimiter to appear between the collapsed"1939b" and "1940a". |
The same issue was discussed in citation-style-language/test-suite#50 and the current behavior seems incorrect. Besides, the grouping and collapsing procedures are also discussed in citation-style-language/schema#338. I check the provided cases by creating a minimal test fixture. One may save it to
I get the result
The output With Finally with |
Here's the full code. Changing the year-suffix-delimiter in the online code editor (Zotero editor does not lat you change the code) has no effect, but if it's normal behavior, I haven't been able to figure out where I went wrong! |
@Circeus Where did you get the citation output I loaded the CSL style file as well as the provided entries and I get the following results in the Zotero style editor:
@Circeus What is your desired output for the style? I'll see if it's possible to implement. Please provide your expected citation for the following case.
|
As I stated above, this is directly from the online editor (https://editor.citationstyles.org/codeEditor/). |
I just find the mistake in #215 (comment)'s fixture: the styles class should be The |
I've been futzing around trying desperately to set a
year-suffix-delimiter
in my style in the online editor. I never managed to, but I figured out why: what actually seems to happen is that when acollapse=
attribute it set,cite-group-delimiter
is used instead ofyear-suffix-delimiter
, which causes the default to be ", " (that ofcite-group-delimiter
) instead of the delimiter set in layout (which is the default foryear-suffix-delimiter
). More importantly, this default will override any explicit values given toyear-suffix-delimiter
,The text was updated successfully, but these errors were encountered: