On Thursday 30 of October 2003 23:45, Leon Bottou wrote: > Incidentally I have another patch for qxembed. > But I do not want to apply it while you are making > the XAddToSaveSet changes. Don't worry, that's why CVS has concurrent in its name. I won't be touching those parts anyway. > > See . I just wondered yesterday why that doesn't work. > The proposed patch is attached to the bug report. > > It reimplements the focus navigation with tab/backtab > using a code that pays more respect to Qt. The new > code makes extensive use of QWidget::focusNextPrevChild() > to search suitable focus windows. Once this is done, > it detects whether we have been looping around and sends > the XEMBED messages as mandated by the spec. > > Please review. All I can say is that it seems to work. -- Lubos Lunak KDE developer --------------------------------------------------------------------- SuSE CR, s.r.o. e-mail: l.lunak@suse.cz , l.lunak@kde.org Drahobejlova 27 tel: +420 2 9654 2373 190 00 Praha 9 fax: +420 2 9654 2374 Czech Republic http://www.suse.cz/