\scalebox with positive scale factor

[aminophen]

A report from Japanese user. When \scalebox is used in math equation, the content is wrongly placed. Here is the MWE and a patch:

%#!pdflatex
\documentclass{article}
\usepackage{amsmath}
\usepackage{graphics}

%%% patch begin
\makeatletter
\long\def\newGscale@box#1[#2]#3{%
  \leavevmode
  \def\Gscale@x{#1}\def\Gscale@y{#2}%
  \setbox\z@\hbox{{#3}}%
  \setbox\tw@\hbox{\Gscale@start\rlap{\copy\z@}\Gscale@end}%
  \ifdim#2\p@<\z@
    \ht\tw@-#2\dp\z@
    \dp\tw@-#2\ht\z@
  \else
    \ht\tw@#2\ht\z@
    \dp\tw@#2\dp\z@
  \fi
  \ifdim#1\p@<\z@
    \hb@xt@-#1\wd\z@{\kern-#1\wd\z@\box\tw@\hss}%
  \else
% change from ...
%    \wd\tw@#1\wd\z@
%    \box\tw@
% ... to
    \hb@xt@#1\wd\z@{\box\tw@\kern#1\wd\z@\hss}%
% can fix the issue
  \fi}
\makeatother
%%% patch end

\begin{document}
\def\contents{\scalebox{2}{a}}

$\frac{\contents}{\contents}$
$\frac{\contents}{\contents\kern1sp}$ % wrong!

\makeatletter
\let\Gscale@box\newGscale@box
\makeatother

$\frac{\contents}{\contents}$
$\frac{\contents}{\contents\kern1sp}$ % correct

\end{document}

[davidcarlisle]

hmm seem like you are right:( Thanks for that, I'll fix before the next release (in the base latex svn, not here) David

[aminophen]

Thanks for that, I'll fix before the next release (in the base latex svn, not here)

Oh, sorry for reporting on wrong repo. Should I re-post this to LaTeX bugs database?

[davidcarlisle]

On 31 March 2017 at 13:43, Hironobu Yamashita ***@***.***> wrote: Thanks for that, I'll fix before the next release (in the base latex svn, not here) Oh, sorry for reporting on wrong repo. Should I re-post this to LaTeX bugs database?

No, here is fine, I get to be the one to fix it either way:-) David

[davidcarlisle]

updated as suggested in latex svn https://www.latex-project.org//svnroot/latex2e-public/trunk/required/graphics/

[aminophen]

Thank you very much!