#3083: Print does not include documentation
-------------------------+--------------------------------------------------
   Reporter:  rlm        |       Owner:  boothby     
       Type:  defect     |      Status:  needs_review
   Priority:  minor      |   Milestone:  sage-4.3.1  
  Component:  notebook   |    Keywords:              
     Author:  Tim Dumol  |    Upstream:  N/A         
   Reviewer:             |      Merged:              
Work_issues:             |  
-------------------------+--------------------------------------------------

Comment(by mpatel):

 Well done!

  * "Closing" a dialog does not remove it from the DOM.  It's just hidden.
 I suggest mapping the close button to, e.g.,
 `$('.dialog').dialog('destroy').remove();`.

  * For `'eval'`-method docstrings, the sans-serif font becomes monospaced.

  * I suggest changing "Click on the box..." to, e.g., "Click here..." and
 binding the click event to just that `div`.  Otherwise, users who click on
 the docstring as they read it and/or copy examples might get an unwanted
 dialog.

  * It might be worth it to mention on the "Help" page or in the tutorial
 which docstrings are printable.

  * If I evaluate `factor?` in a cell, I can print the docstring with the
 worksheet, as desired.  People might also find it useful for evaluated
 docstrings to survive reopening the worksheet or to appear in published
 worksheets (e.g., as part of an annotated tutorial).  But I suggest
 waiting for feedback on the mailing lists and, if it's necessary, creating
 new tickets.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/3083#comment:5>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica, 
and MATLAB
-- 
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.


Reply via email to