#5050: [with patch, needs review] clean up how the percent directives at the top
of cells are processed
----------------------+-----------------------------------------------------
 Reporter:  mhansen   |        Owner:  boothby 
     Type:  defect    |       Status:  new     
 Priority:  major     |    Milestone:  sage-3.3
Component:  notebook  |   Resolution:          
 Keywords:            |  
----------------------+-----------------------------------------------------
Changes (by jason):

  * summary:  clean up how the percent directives at the top of cells are
              processed => [with patch, needs review] clean
              up how the percent directives at the top of
              cells are processed

Comment:

 One positive review.  It's probably important enough that at least one
 other person ought to review it as well.

 Mike and I stepped through the changes, they all look good, and I tested
 for about 20 minutes with various combinations of the directives and
 things seemed to work.  I had one glitch, but I can't reproduce it.  I had
 a cell that looked like:

 {{{
 %hide
 %hideall
 print "hi"
 }}}

 I closed the cell, reopened it, then deleted all output, then did a few
 more things I can't remember.  At one point, the cell changed to:

 {{{
 %hide
 %hid
 }}}

 If we can't reproduce something like this, though, I give it a positive
 review.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5050#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
-~----------~----~----~----~------~----~------~--~---

Reply via email to