On Fri, 17 Aug 2007, Abdelrazak Younes wrote:

The patch itself is not that big. The changes in BufferView is all about removal of function and removal of test of the type 'if (buffer_)' which does not make sense anymore.

So, objection?

Thanks for the updated description, I may even dare to try and look at the patch/code this weekend now:-) As it's Friday evening after a long day,
I'd like to ask if I've understood the description reasonably correct.

Ignoring Qt specializations.

  LyXView
    |
    <tab-widget>
       | (many)
       WorkArea <-----------> Buffer
         |
         BufferView
           |
           Cursor

Don't know how to represent that there may be many WorkAreas in the tab widget, or that there may be more buffers in the system. Anyway, if the image is reasonably accurate, I think it might help to be placed with the descriptive text.

Should there be an 'Application' somewhere in there as well?

/C

--
Christian Ridderström, +46-8-768 39 44               http://www.md.kth.se/~chr

Reply via email to