search

latex3 / latex2e

The LaTeX2e kernel
https://www.latex-project.org/
LaTeX Project Public License v1.3c
1.91k stars 264 forks source link

Outdated definition of T1 \k (in example) #1148

Closed ivankokan closed 4 months ago

ivankokan commented 1 year ago

https://github.com/latex3/latex2e/blob/de8ca4a2b3a982055a7ccee9556901024d2a2c98/base/doc/fntguide.tex#L1462-L1467

The definition is not up-to-date. Not really important but still...

On top of this, I was looking at L7x which uses the former definition:

\DeclareTextCommand{\k}{L7x}[1]{\oalign{\null#1\crcr\hidewidth\char12}}

Should the L7x maintainers consider updating their definition as well?

FrankMittelbach commented 1 year ago

Can't remember the reason why we changed to \hmode@bgroupall over the place, but it must have been because of some bug. So yes I think any other encodings should follow too.

ivankokan commented 1 year ago

True indeed: a0c7180, the initial commit here already uses \hmode@bgroup.

ivankokan commented 12 months ago

Can't remember the reason why we changed to \hmode@bgroupall over the place, but it must have been because of some bug. So yes I think any other encodings should follow too.

It seems that many other encodings are affected (T2A, T2B, T2C, but the list goes on - based on my complete TeX Live 2023 installation), and it might be that other commands (not only \k) should be treated as well.

Not sure how to proceed with this, it might be tedious to address all maintainers individually (@yannis1962 for L7x is one of them, will this mention reach him?).

FrankMittelbach commented 11 months ago

I'll keep this open because at some point we want to fix the T2 encodings too ... but not for this release.

FrankMittelbach commented 11 months ago

Not sure how to proceed with this, it might be tedious to address all maintainers individually (@yannis1962 for L7x is one of them, will this mention reach him?).

This is tedious, I know. But if these things should get fixed that is about the only way to proceed. I have now done it for Cyrillic (will show up in the next release), but I'm not going to look after other third-party encodings --- too many other topics to look after. And we can't really track them here either.

github-actions[bot] commented 7 months ago

This issue has been automatically marked as stale because it has not had recent activity.