#5433: [with patch, review pending] LaTeX fixes
------------------------+---------------------------------------------------
Reporter: jhpalmieri | Owner: jhpalmieri
Type: defect | Status: new
Priority: minor | Milestone: sage-3.4.2
Component: misc | Keywords:
------------------------+---------------------------------------------------
Comment(by rbeezer):
Replying to [comment:1 jhpalmieri]:
Hi John,
Working on a review. Looks good and all tests passed.
There was a time when TeX would choke on long input (more than 256
characters?) so maybe that explains the use of "brk"? But I think those
days are behind us, so it is probably safe to drop it.
Two suggestions:
1. In the docstring for {{{_latex_file_()}}} it is not real clear that
the {{{sep}}} argument is for decoration rather than function. Maybe
additional explanation like: "for example, {{{sep='\\hrule'}}}", would
make it clearer that this is not a LaTeX requirement like "$$" (and it
also needs an escaped backslash for any TeX commands). (Which I now see
is discussed in #3145.)
2. Are all the "empty" {{{\pagestyle}}} and {{{\thispagestyle}}} really
needed? There is one present in the header, but then two show up between
each object's latex representation. They don't seem to be hurting
anything, but maybe now would be a good time to have them go away, perhaps
along with a {{{\n}}} or two.
Rob
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5433#comment:2>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of
Reinventing the Wheel
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to
[email protected]
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en
-~----------~----~----~----~------~----~------~--~---
