reutenauer / polyglossia

An alternative to Babel for XeLaTeX and LuaLaTeX
http://www.ctan.org/pkg/polyglossia
MIT License
190 stars 51 forks source link

A fix for #608 #609

Closed Udi-Fogiel closed 10 months ago

Udi-Fogiel commented 10 months ago

I'm not sure what was the intention in https://github.com/reutenauer/polyglossia/commit/d3027374d0ac2f3ddc00b067292d23983a32a9dd , but the comment says that the language options should be lowered case, not the language tag, so I think this is the correct fix.

Udi-Fogiel commented 10 months ago

oh, never mind, that defeats the whole purpose of https://github.com/reutenauer/polyglossia/commit/d3027374d0ac2f3ddc00b067292d23983a32a9dd ...

Udi-Fogiel commented 10 months ago

maybe using \AddToNoCaseChangeList would solve both problems?

Udi-Fogiel commented 10 months ago

Ok, now I think it solve both problems.

Udi-Fogiel commented 10 months ago

An explanation can be found in interface3.pdf, section 33.4:

\l_text_case_exclude_arg_tl: Lists commands where the first mandatory argument is excluded from case changing.

jspitz commented 10 months ago

I resolved the typo myself. Thanks, @Udi-Fogiel

Udi-Fogiel commented 10 months ago

I resolved the typo myself. Thanks, @Udi-Fogiel

Thanks!