Opened 12 years ago

Closed 12 years ago

#7117 closed defect (fixed)

[with patch, positive review] latex fix for RIF

Reported by: jhpalmieri Owned by: cwitty
Priority: major Milestone: sage-4.2
Component: misc Keywords:
Cc: Merged in: sage-4.2.alpha0
Authors: John Palmieri Reviewers: Mike Hansen
Report Upstream: Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description

From IRC:

By the way, if I evaluate "jsmath(RIF)" in the notebook, jsMath complains: "Unknown control sequence '\I'". 
Is there a missing macro definition?

It looks to me as though the _latex_ method for RIF has been defined in terms of '
I' for a long time, and it has not worked since at least Sage 3.4. The attached patch changes it from "
I
R" to "
Bold{I}
Bold{R}".

Attachments (1)

trac_7117-rif.patch (2.2 KB) - added by mhansen 12 years ago.

Download all attachments as: .zip

Change History (3)

comment:1 Changed 12 years ago by mhansen

  • Reviewers set to Mike Hansen
  • Summary changed from [with patch, needs review] latex fix for RIF to [with patch, positive review] latex fix for RIF

Looks good to me.

Changed 12 years ago by mhansen

comment:2 Changed 12 years ago by mhansen

  • Merged in set to sage-4.2.alpha0
  • Resolution set to fixed
  • Status changed from positive_review to closed

I had to do a minor rebasing and attached the new patch.

Note: See TracTickets for help on using tickets.