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

Change \capitalisewords to \ecapitalisewords in the documentation #262

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jonasc
Copy link

@jonasc jonasc commented Oct 17, 2023

I tried to have title casing for a command and followed the suggestion in the documentation to use \capitalisewords from mfirstuc. However, it doesn't work as expected, as it doesn't change anything. Some example I found in issue #86 (comment) also does not work (anymore?).

However, after thinking and fiddling around, I saw that mfirstuc also has a command \ecapitalisewords which expands the provided argument before capitalizing it. And it actually works. Thus, to save future readers the hassle of having to find it out on their own, I propose to change the command referenced in the documentation to \ecapitalisewords.

Using `\capitalisewords` does not actually work, but `\ecapitalisewords` does.
@jonasc
Copy link
Author

jonasc commented Oct 17, 2023

Here is an example showing what works and what not:

\documentclass{article}
\usepackage{acro,mfirstuc}

\DeclareAcronym{foobar}{
  short = funny objective obsolete BAR,
  long = funny objective obsolete business around rice
}

\begin{document}
  \paragraph{\texttt{ecapitalisewords}}
  \Acl[uppercase/cmd=\ecapitalisewords]{foobar} is title cased.
  \Acs[uppercase/cmd=\ecapitalisewords]{foobar} is title cased.
  \par
  \paragraph{\texttt{capitalisewords}}
  \Acl[uppercase/cmd=\capitalisewords]{foobar} is NOT title cased.
  \Acs[uppercase/cmd=\capitalisewords]{foobar} is NOT title cased.
\end{document}

2023-10-17T10:06:52,254762213+02:00

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

1 participant