-
Notifications
You must be signed in to change notification settings - Fork 98
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
Framed fix #2161
Framed fix #2161
Conversation
…Positionable.attributes) where they make more sense
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks healthy! I left a couple of minor comments, but nothing significant.
@@ -4695,7 +4695,7 @@ DefMacro('\framebox', '\@ifnextchar(\pic@framebox\@framebox', robust => 1); | |||
DefConstructor('\@framebox[Dimension][]{}', | |||
"?#mathframe(<ltx:XMArg enclose='box'>#inner</ltx:XMArg>)" | |||
. "(<ltx:text ?#width(width='#width') ?#align(align='#align')" | |||
. " framed='rectangle' framecolor='#framecolor'" | |||
. " framed='rectangle' framecolor='#framecolor' cssstyle='#cssstyle'" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am spiritually obliged to mention css_style
looks better on every PR that uses the non-underscore variant :>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
css_style
??? so boring...
<td align="left"><text backgroundcolor="#FF0000" class="ltx_phantom" font="sansserif" fontsize="90%" framecolor="black" framed="rectangle">XX</text><text font="sansserif" fontsize="90%"> 0 1 1</text></td> | ||
<td align="left"><text backgroundcolor="#FF0000" class="ltx_phantom" font="sansserif" fontsize="90%" framecolor="black" framed="rectangle">XX</text><text font="sansserif" fontsize="90%"> FF0000</text></td> | ||
<td align="left"><text backgroundcolor="#4D4D4D" class="ltx_phantom" font="sansserif" fontsize="90%" framecolor="black" framed="rectangle">XX</text><text font="sansserif" fontsize="90%"> 0.3</text></td> | ||
<td align="left"><text backgroundcolor="#FF0000" class="ltx_phantom" cssstyle="padding:0.0pt" font="sansserif" fontsize="90%" framecolor="black" framed="rectangle">XX</text><text font="sansserif" fontsize="90%"> 1 0 0</text></td> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Quickly marking that the test using ltx_phantom
text now gets explicit 0 padding.
Probably intentional?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think so.
This PR uses the
framed
attribute, vsltx_framed
CSS a bit more consistently, allows for adjusting the frame separation, and avoids collapsing nodes which have framing.Fixes #2134
Fixes #2135