Ticket #7117 (closed defect: fixed)

Opened 4 years ago

Last modified 4 years ago

[with patch, positive review] latex fix for RIF

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

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

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

Change History

comment:1 Changed 4 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 4 years ago by mhansen

comment:2 Changed 4 years ago by mhansen

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

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

Note: See TracTickets for help on using tickets.