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 missing % in apaauthor #88

Merged
merged 1 commit into from
Feb 1, 2020
Merged

Add missing % in apaauthor #88

merged 1 commit into from
Feb 1, 2020

Conversation

moewew
Copy link
Contributor

@moewew moewew commented Jan 31, 2020

Adds a missing %s in apaauthor and normalises the indentation.


The usage of \printtext within a name format can cause quite undesirable effects, so I would recommend to get rid of them. Unfortunately, that means that you can't use \setunit and friends either and have to check for empty contents manually.

Example of undesirable effect

\documentclass[british]{article}
\usepackage[T1]{fontenc}
\usepackage[utf8]{inputenc}
\usepackage{babel}
\usepackage{csquotes}

\usepackage[style=apa, backend=biber]{biblatex}

\DeclareDelimFormat*{multinamedelim}{\slash} 
\DeclareDelimFormat*{finalnamedelim}{} 
\DeclareDelimAlias{finalnamedelim}{multinamedelim} 
\DeclareDelimFormat*{finalnamedelim:apa:family-given}{}
\DeclareDelimAlias{finalnamedelim:apa:family-given}{multinamedelim}

\addbibresource{biblatex-examples.bib}

\begin{document}
\cite{sigfridsson}
\printbibliography
\end{document}

@plk plk merged commit a7cde62 into plk:master Feb 1, 2020
@moewew moewew deleted the apaauthor branch February 1, 2020 10:33
moewew added a commit to moewew/biblatex-apa that referenced this pull request Feb 1, 2020
\printtext and \setunit *within* a name format or field format
can confuse the punctuation tracker.
This required some manual intervention, but should avoid problems
as the one demonstrated in plk#88.
moewew added a commit to moewew/biblatex-apa that referenced this pull request Feb 22, 2020
@moewew moewew mentioned this pull request Feb 22, 2020
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.

2 participants