#7254: [with patch; needs review] sagenb notebook: automatic worksheet 
refreshing
and synchronization
---------------------------+------------------------------------------------
   Reporter:  was          |       Owner:  boothby   
       Type:  enhancement  |      Status:  needs_work
   Priority:  major        |   Milestone:  sage-4.2  
  Component:  notebook     |    Keywords:            
Work_issues:               |      Author:            
   Reviewer:               |      Merged:            
---------------------------+------------------------------------------------
Changes (by mpatel):

  * status:  needs_review => needs_work


Comment:

 Also:

  * It's probably safer to use `parseInt` and to use `===` and `!==`,
 instead of `==` and `!=`, to avoid JS's implicit conversions (cf.
 [http://www.jslint.com/ JSLint's] "The Good Parts" setting).  Although I'm
 not sure how it happened, I saw the successive state "numbers" 480, 4801,
 48011, etc.
  * The browser and server do not update their `state_number`s if a new
 cell is added at the end of a sheet, when text cells are added, or cells
 are deleted.  Should we add these to the list, along with the
 `'delete_all_output'`, `'pretty_print'`, `'system'` worksheet commands?

 If these changes are worthwhile, I can make a patch, but I'm not sure
 how/where to store the worksheet's state (or "revision") number.

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