[XeTeX] Problem with the bidi package: chapter titles flushed right

[Ross Moore]

Hi Vafa,

On 20/08/2009, at 1:47 AM, Vafa Khalighi wrote:

> Thanks for reporting. Put this after loading bidi package and  
> before \begin{document}:
>
> %%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
> \makeatletter
> \newcommand\bidiraggedleft{\if at RTL\raggedright\else\raggedleft\fi}
> \newcommand\bidiraggedright{\if at RTL\raggedleft\else\raggedright\fi}
> \def\@makechapterhead#1{%
>   \vspace*{50\p@}%
>   {\parindent \z@ \if at RTL\bidiraggedright\else \normalfont

This code block is unbalanced, by missing a \fi .
Do you really need \if at RTL here, as that test is performed
anyway inside \bidiraggedright ?
As you have it, the \normalfont  will not be executed in RTL.

>     \ifnum \c at secnumdepth >\m at ne
>         \huge\bfseries \@chapapp\space \thechapter
>         \par\nobreak
>         \vskip 20\p@
>     \fi
>     \interlinepenalty\@M
>     \Huge \bfseries #1\par\nobreak
>     \vskip 40\p@
>   }}
> \def\@makeschapterhead#1{%
>   \vspace*{50\p@}%
>   {\parindent \z@ \bidiraggedright
>     \normalfont
>     \interlinepenalty\@M
>     \Huge \bfseries  #1\par\nobreak
>     \vskip 40\p@
>   }}
> \makeatother
> %%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%5
>
> Best Wishes,
> Vafa Khalighi


Cheers,

	Ross

------------------------------------------------------------------------
Ross Moore                                       ross at maths.mq.edu.au
Mathematics Department                           office: E7A-419
Macquarie University                             tel: +61 (0)2 9850 8955
Sydney, Australia  2109                          fax: +61 (0)2 9850 8114
------------------------------------------------------------------------