Re: Warnings when using Listings in a RTL document

2023-03-22 Thread Udicoudco
On Wed, Mar 22, 2023 at 10:28 PM Udicoudco wrote: > > > I attached a possible patch, it only fix the lstlisting case when > > polyglossia is used, since babel-hebrew does not have an equivalent > > to \begin{RTL}...\end{RTL} (or there is one?) and babel with luatex > > does not need this kind of w

spurious spaces around forceLTR insets

2023-03-22 Thread Udicoudco
Hello all, In some cases, e.g. displayed equation, LyX wraps a forceLTR inset like so: \LRE{ \[ a+b=c \] } continuous text here... which might cause undesired space if the user is not careful. Attached a LyX file to demonstrate the problem and a patch to fix it. Regards, Udi diff --git a/src/P

Re: Warnings when using Listings in a RTL document

2023-03-22 Thread Udicoudco
> I attached a possible patch, it only fix the lstlisting case when > polyglossia is used, since babel-hebrew does not have an equivalent > to \begin{RTL}...\end{RTL} (or there is one?) and babel with luatex > does not need this kind of wrappers. I've added line breaks to the LaTeX code for readabi

Re: Warnings when using Listings in a RTL document

2023-03-22 Thread Udicoudco
On Wed, Mar 22, 2023 at 2:22 PM Scott Kostyshak wrote: > > On Wed, Mar 22, 2023 at 01:13:27PM +0100, Jürgen Spitzmüller wrote: > > Am Mittwoch, dem 22.03.2023 um 12:54 +0200 schrieb Udicoudco: > > > I think in the case of where forceLTR insets will have multiple > > > paragraphs, it is better to u

Re: Warnings when using Listings in a RTL document

2023-03-22 Thread Scott Kostyshak
On Wed, Mar 22, 2023 at 01:13:27PM +0100, Jürgen Spitzmüller wrote: > Am Mittwoch, dem 22.03.2023 um 12:54 +0200 schrieb Udicoudco: > > I think in the case of where forceLTR insets will have multiple > > paragraphs, it is better to use > > \begin{RTL}...\end{RTL}, which basically only issues \par a

Re: Warnings when using Listings in a RTL document

2023-03-22 Thread Jürgen Spitzmüller
Am Mittwoch, dem 22.03.2023 um 12:54 +0200 schrieb Udicoudco: > I think in the case of where forceLTR insets will have multiple > paragraphs, it is better to use > \begin{RTL}...\end{RTL}, which basically only issues \par and sets > the > RTL related conditionals > to be false. Thanks for the deta

Re: Warnings when using Listings in a RTL document

2023-03-22 Thread Udicoudco
Slight correction, the lines > > \@RTLfalse\beginL{ > \begin{lstlisting} > Hello > \end{lstlisting} > }\endL > should be {\@RTLfalse\beginL \begin{lstlisting} Hello \end{lstlisting} }\endL and the lines > > \@RTLfalse\beginL{ > \par > Hello > \par > }\endL > should be {\@RTLfalse\beginL

Re: Warnings when using Listings in a RTL document

2023-03-22 Thread Udicoudco
On Tue, Mar 21, 2023 at 9:40 PM Scott Kostyshak wrote: > > On Tue, Mar 21, 2023 at 12:50:06PM +0100, Jürgen Spitzmüller wrote: > > Am Dienstag, dem 21.03.2023 um 11:27 + schrieb Udicoudco: > > > Hello all, > > > > > > Attached a LyX file and its export to LaTeX code. > > > When compiling with