On Sunday, December 22, 2013 2:05:08 PM UTC-6, Edward K. Ream wrote: > > This implements the ideas in the thread: > https://groups.google.com/forum/#!topic/leo-editor/Wd0NtBLEZZ<https://groups.google.com/forum/#!topic/leo-editor/Wd0NtBLEZZk> >
This project has been a great success. Rev 6465 smooths all the rough edges mentioned in this thread: - All code can call p.isExpanded as before. This is an important guarantee of compatibility. This one change removes most of the danger from this project. - p.expand and p.contract now optimally manage the expandedPositions list. This means that contracting a node does not automatically contract all its clones. This is a much more intuitive mode of operation than before. - As a side project, I rewrote and simplified p.isVisible so it no longer uses the evil c.visLimit method. The latter method obscures all the code in which it appears. Note that moving a node will, in all likelihood, contract most clones of that node because the saved positions become invalid. This is baked into the scheme, and I have no intention of trying to "fix" this. I have more experience with the code, and I think it is solid. However, please do look out for problems and report any such immediately. Thanks. Edward -- You received this message because you are subscribed to the Google Groups "leo-editor" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. Visit this group at http://groups.google.com/group/leo-editor. For more options, visit https://groups.google.com/groups/opt_out.
