-
Notifications
You must be signed in to change notification settings - Fork 1
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
Allow bidi to be loaded early #29
Comments
Thanks for the suggestion. This is quite tricky and it needs a lot of testing to make sure that it does not break anything since the From memory, I was in the past 10 years was thinking of doing something similar; at the time, the package Having said that, I willl look into this when I have time and will release an experimental version so people can test. |
It should not affect old documents, as all the current documents that use |
But my first suggestion wasn't considering catcodes: \newcommand*\bidi@restore@catcode{}
\newcommand\bidi@isloaded[1]{
\@ifpackageloaded{#1}%
{\input{#1-xetex-bidi.def}}%
{\AddToHook{package/#1/after}{
\edef\bidi@restore@catcode{\catcode`\@=\the\catcode`\@}
\makeatletter
\input{#1-xetex-bidi.def}
\bidi@restore@catcode
}
}
} |
It is not that simple. |
I know you are currently very busy, but if you can elaborate a little bit more, I'll be happy to learn how could that potentially affect already existing documents. |
Brief outline of the feature request
Check/indicate
xepersian
packagebidi
package issue tracker](https://github.com/kvafa/bidi/issues) has been searched for similar issues?xepersian
package issue tracker has been searched for similar issues?Today, with LaTeX's new hook system it is possible to define
\bidi@isloaded
in such a way thatbidi
could be load in any order with respect to other packages. one option could beand removing the the begindocument hook from
lodingorder-xetex-bidi.def
and also removing the loading of the .def files from ltx-xetex-bidi.def (or moving \bidi@isloaded to there).This is just a suggestion, I'm not sure if anything else would also be needed, but I hope this demonstrate the idea, it could make things much simpler...
The text was updated successfully, but these errors were encountered: