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.

Sign up for GitHub

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

Jump to bottom

\__fontspec_check_script:Nn fails in some cases #530

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from
Open

\__fontspec_check_script:Nn fails in some cases #530

wants to merge 3 commits into from

Commits on Sep 6, 2024

  1. \__fontspec_check_script:Nn fails in some cases 

    If for some reason \curr@fontshape is not the really
the shape of the currently selected font, e.g. due
to a missing shape, and LuaTeX is the used engine,
the expansion of '\curr@fontshape/\f@size' is not
the name of the font switch that needs to be checked.
Example:

\documentclass{article}
\usepackage{fontspec}
\setmainfont{David CLM}
\begin{document}
\scshape % comment this line to see the effect
\ExplSyntaxOn
\fontspec_if_script:nTF{hebr}{yes}{no}
\end{document}

Note how this example returns true with LuaTeX but
false with XeTeX.

It would be more reliable to use \the\font to get
the name of the switch of the currently selected font.
    @Udi-Fogiel
    Udi-Fogiel committed Sep 6, 2024
    Configuration menu
    Copy the full SHA
    777bcf1 View commit details
    Browse the repository at this point in the history

  2. add regression test for latex3#530

    @Udi-Fogiel
    Udi-Fogiel committed Sep 6, 2024
    Configuration menu
    Copy the full SHA
    39e550d View commit details
    Browse the repository at this point in the history

  3. use a font listed in tl_packages in api-script-missing-shape.lvt

    @Udi-Fogiel
    Udi-Fogiel committed Sep 6, 2024
    Configuration menu
    Copy the full SHA
    c80e79c View commit details
    Browse the repository at this point in the history