On Sun, Mar 09, 2008 at 11:23:51PM -0000, [EMAIL PROTECTED] wrote:

> Author: uwestoehr
> Date: Mon Mar 10 00:23:50 2008
> New Revision: 23608
> 
> URL: http://www.lyx.org/trac/changeset/23608
> Log:
> InsetMathNest.cpp: Ctrl-b uses now \boldsymbol (fixes 
> http://bugzilla.lyx.org/show_bug.cgi?id=499)

This breaks bold super and subscripts. Please revert.

\boldsymbol uses \mbox, thus preventing the use of the correct size.
Moreover, the output of \boldsymbol{c} is different from \mathbf{c}.
\boldsymbol is to be used only in certain circumstances and should
not be easily and unknowingly obtainable as a substitute for \mathbf.
I think that bug 499 should be marked as WONTFIX. If you need a bold
greek letter, you can type \boldsymbol in mathed. Of course, the
documentation should also warn about the consequences of using \boldsymbol.

> Modified:
>     lyx-devel/trunk/src/mathed/InsetMathNest.cpp
> 
> Modified: lyx-devel/trunk/src/mathed/InsetMathNest.cpp
> URL: 
> http://www.lyx.org/trac/file/lyx-devel/trunk/src/mathed/InsetMathNest.cpp?rev=23608
> ==============================================================================
> --- lyx-devel/trunk/src/mathed/InsetMathNest.cpp (original)
> +++ lyx-devel/trunk/src/mathed/InsetMathNest.cpp Mon Mar 10 00:23:50 2008
> @@ -808,7 +808,7 @@
>               if (currentMode() == TEXT_MODE)
>                       handleFont(cur, cmd.argument(), "textbf");
>               else
> -                     handleFont(cur, cmd.argument(), "mathbf");
> +                     handleFont(cur, cmd.argument(), "boldsymbol");
>               break;
>       case LFUN_FONT_SANS:
>               if (currentMode() == TEXT_MODE)
> 

-- 
Enrico

Reply via email to