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

lettrine-xetex-bidi.def outdated #31

Open
4 tasks
dflipo opened this issue Oct 2, 2023 · 4 comments
Open
4 tasks

lettrine-xetex-bidi.def outdated #31

dflipo opened this issue Oct 2, 2023 · 4 comments
Assignees
Labels
bug Something isn't working

Comments

@dflipo
Copy link

dflipo commented Oct 2, 2023

Brief outline of the issue

lettrine-xetex-bidi.def patches an old definition of the internal command @lettrine of the lettrine package leading to issues with the current implementation.

Rather than updating the patched definition in lettrine-xetex-bidi.def, I suggest to include the RTL flag mechanism for the \parshape command in the original definition of @lettrine. I have an experimental version of lettrine.sty, appended here. Please let me know
lettrine.sty.gz
if something like that would enable to get rid of lettrine-xetex-bidi.def?

Check/indicate

Minimal example showing the issue

% !TEX TS-program = XeLaTeX
% !TEX encoding = UTF-8 Unicode


\documentclass{article}            % or some other class

  % Any packages other than the bidi package must be loaded here
\usepackage{lettrine}

  % The bidi package must be loaded as the last package
\usepackage{bidi}

  % Any preamble code goes here
  
\begin{document}

 \lettrine{V}{oici} un exemple 

\end{document}

Expected behavior

Log and PDF files

! Undefined control sequence.
\LettrineFont \L@initial

@dflipo dflipo added the bug Something isn't working label Oct 2, 2023
@kvafa
Copy link
Owner

kvafa commented Oct 28, 2023

Thanks for your bug report. Sure, I will certainly look into this but I'm afraid that will not be anytime soon as I am occupied by other things currently.

@EliseyP
Copy link

EliseyP commented Mar 21, 2024

May be this post is answer?

#13 (comment)

This lettrine-xetex-bidi.def in texmf-local/tex/xelatex/bidi solve this problem for me.

@dflipo
Copy link
Author

dflipo commented Mar 21, 2024

NO, this file is incompatible with the current version of lettrine.sty [2024/01/20 v.2.60]. Lettrine.sty is aware of bidi now.
Please consider replacing lettrine-xetex-bidi.def by the appended (zipped) version.
lettrine-xetex-bidi.zip

Patching an internal command of lettrine.sty (which is liable to change) is definitely not the way to go.

@EliseyP
Copy link

EliseyP commented Mar 23, 2024

Thank you!
Yes, it was some kind of "dirty hacking" - I had to compile my task urgently.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants