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

Add non-breaking space to AD/BC terms in de-DE locale #148

Merged
merged 1 commit into from Dec 17, 2016

Conversation

tillheilmann
Copy link
Contributor

Added (non-breaking) space to terms "ad" and "bc"

Added (non-breaking) space to terms "ad" and "bc"
@csl-bot
Copy link
Contributor

csl-bot commented Dec 14, 2016

Awesome! You've created a pull request to the Citation Styles Language locales repository. We'll get in touch soon (usually within a day or two). In the meantime, our automated test system will go ahead and run some checks on your pull request. In a few minutes you'll be notified of the test results.

If you haven't done so yet, please make sure your locale file validates.

To update the current pull request, visit the "Files changed" tab above, and click on the pencil icon (see below) in the top-right corner of your locale file to start editing.

image

If you need assistance at any point, please leave a comment and we'll get back to you (feel free to write in Dutch, English, French, German, Portuguese, or Spanish).

@csl-bot
Copy link
Contributor

csl-bot commented Dec 14, 2016

Yay! Your pull request passed all our automated tests. We'll take a look soon.

@adam3smith
Copy link
Member

adam3smith commented Dec 14, 2016

@rmzelle -- just want to confirm with you that you agree this is right in the locale (I told Till it is): The English 40bc is 40 v. Chr. in German, so it makes sense to me to code the space into the term.

@rmzelle
Copy link
Member

rmzelle commented Dec 14, 2016

@adam3smith, do most style guides not use a separating space? E.g. https://en.wikipedia.org/wiki/Anno_Domini does ("AD 525" and "1 BC")

@tillheilmann, have you confirmed that this change actually works? (in Zotero, Mendeley, or something else)

@adam3smith
Copy link
Member

you're right. Chicago Manual adds a space, too. @fbennett do you have any recollection on why citeproc-js doesn't add a space between year and BC/AD terms?

@rmzelle
Copy link
Member

rmzelle commented Dec 14, 2016

do you have any recollection on why citeproc-js doesn't add a space between year and BC/AD terms?

Maybe because we wrote it that way in the spec? :)

http://docs.citationstyles.org/en/stable/specification.html#ad-and-bc says

The “ad” term (Anno Domini) is automatically appended to positive years of less than four digits (e.g. “79” becomes “79AD”). The “bc” term (Before Christ) is automatically appended to negative years (e.g. “-2500” becomes “2500BC”).

As long as the CSL processors don't trim white-space characters defined in the term definitions, this is probably the right solution, since it allows for the choice between regular and non-breaking delimiting spaces, as proposed in this PR.

@adam3smith
Copy link
Member

good point -- we're taking these then, yes?

@rmzelle
Copy link
Member

rmzelle commented Dec 14, 2016

Could wait for Frank to chime in, but otherwise yes.

@tillheilmann
Copy link
Contributor Author

@rmzelle, the change of terms (with added non-breaking spaces) works for me if I redefine the terms in the header of my custom CSL files and produce output with Zotero (4.0) on my Mac. I have not tried other versions/programs/OSs.

@rmzelle rmzelle merged commit c7a5209 into citation-style-language:master Dec 17, 2016
@rmzelle
Copy link
Member

rmzelle commented Dec 17, 2016

Thanks!

@rmzelle rmzelle changed the title Update locales-de-DE.xml Add non-breaking space to AD/BC terms in de-DE locale Dec 17, 2016
@tillheilmann tillheilmann deleted the patch-9 branch October 10, 2017 07:24
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.

None yet

4 participants