Opened 13 years ago

Closed 12 years ago

Last modified 12 years ago

#3009 closed defect (fixed)

[with patch, positive review] remove spaces after continuation characters "\"

Reported by: mabshoff Owned by: jwmerrill
Priority: major Milestone: sage-3.1.2
Component: documentation Keywords:
Cc: Merged in:
Authors: Reviewers:
Report Upstream: Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description

John H Palmieri reports:

In the section of the tutorial on 3d plotting,

<http://sagemath.org/doc/html/tut/node22.html>

the continuation characters "\" all have spaces after them, which messes up cutting and pasting. Also in the section on Maxima,

<http://sagemath.org/doc/html/tut/node54.html>

two of the backslashes have spaces after them (in the Mobius strip and the Klein bottle examples).

The same thing happens half a dozen times in "SAGE Constructions".

I've only searched the tutorial and the constructions documentation for this issue (by searching the files "tut.tex" and "const.tex"); I haven't looked at the rest of the documentation.

Attachments (2)

3009_docstrings.patch (1.7 KB) - added by jwmerrill 12 years ago.
3009_tex.patch (3.3 KB) - added by jwmerrill 12 years ago.

Download all attachments as: .zip

Change History (6)

Changed 12 years ago by jwmerrill

Changed 12 years ago by jwmerrill

comment:1 Changed 12 years ago by jwmerrill

  • Owner changed from tba to jwmerrill
  • Summary changed from remove spaces after continuation characters "\" to [with patch, needs review] remove spaces after continuation characters "\"

I did a regex search on all the tex documentation and all the docstrings for the pattern "
[\ ]+$" and replaced all the appropriate matches (filtered by eye). The changes are separated into patches for the docstrings and patches for the tex documentation. If this is accepted, both patches should be applied.

comment:2 Changed 12 years ago by mabshoff

  • Summary changed from [with patch, needs review] remove spaces after continuation characters "\" to [with patch, positive review] remove spaces after continuation characters "\"

Patches look good to me. Positive review.

Cheers,

Michael

comment:3 Changed 12 years ago by mabshoff

  • Resolution set to fixed
  • Status changed from new to closed

Merged in Sage 3.1.2.rc3

comment:4 Changed 12 years ago by mabshoff

  • Milestone changed from sage-3.1.3 to sage-3.1.2
Note: See TracTickets for help on using tickets.