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

Do not blindly lowercase language names #13

Closed
wants to merge 56 commits into from
Closed

Do not blindly lowercase language names #13

wants to merge 56 commits into from

Conversation

jspitz
Copy link
Contributor

@jspitz jspitz commented Apr 24, 2017

Rather than that, only do that if they are completely uppercased
(which is probably the cause of a \MakeUppercase e.g. in headings)

This assures miced-case names such as USenglish are treated correctly

Fixes issue #12.

Rather than that, only do that if they are completely uppercased
(which is probably the cause of a \MakeUppercase e.g. in headings)

This assures miced-case names such as USenglish are treated correctly

Fixes issue #12.
@jspitz
Copy link
Contributor Author

jspitz commented May 2, 2017

Thanks for the feedback. I thought the braces would group the second token in the test and my tests seem to have acknowledged that, but I stand corrected. I figure that the following would work, right?

+\def\csq@checkuc#1{%
+ \def\tempa{#1}%
+ \def\tempb{\MakeUppercase{#1}}%
+  \ifx\tempa\tempb
+     \lowercase{#1}%
+  \else
+     #1%
+  \fi}

But your last sentence indicates that you do not like the solution at all. Alas, I don't have a better idea, it's as general as it gets.

josephwright and others added 27 commits June 22, 2019 17:51
Use polygossia's new babel-equivalent language name interface.
Only works with polyglossia 1.45 (2019/20/27) or above,
but the code is fully backwards compatible with older versions
(variant detection will fail then).

See also #4.
Improve polyglossia interface
\languagename may not always be available, so we better test for it.
Furthermore, \languagename and \babelname should not be expanded
for the \ifx test.
\langname does not exist.
Fixes #30.
Set up rudimentary tests
@jspitz
Copy link
Contributor Author

jspitz commented Dec 4, 2019

This PR seems screwed. I'll open a new one.

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.

8 participants