On Thu, 05 Oct 2000, Allan Rae wrote:
> This should be an updateBufferDependent() and the insets should hide
> themselves if appropriate.  I did this in the old tree and it worked fine.
> I'll commit this as it is at present however (once I've looked through and
> test compiled of course).

Allan, I've read your comments, here, in the ChangeLog and in 
BufferView_pimpl.C. 

As I understand it, you're suggesting that each FormXXX would store a 
Buffer * dialogBuffer. This would be set to 0 when the dialog was hidden, but 
would be set to the current buffer in show(). update() would check that 
(dialogBuffer == lv->buffer()) and, if not, would call hide().

Is this the idea?

I have one final question. In BufferView_pimpl.C, your new 
updateBufferDependent() signal occurs before the

        if (!buffer_) {
                buffer_ = bufferlist.first();
        }

Shouldn't the signal be emitted immediately after this call? Finally, the 
extra updateBufferDependent() signal in the next if( buffer ) loop is now 
redundant?

Shall I have a go at all this?
Angus

Reply via email to