Am 21.03.2011 um 18:00 schrieb Stephan Witt: > Am 21.03.2011 um 09:43 schrieb Abdelrazak Younes: > >> On 03/21/2011 07:30 AM, Stephan Witt wrote: >>> Am 20.03.2011 um 17:50 schrieb Abdelrazak Younes: >>> >>>> Hi Pavel, >>>> >>>> Here is a patch that should fixes #7375. The patch is big but this is >>>> mostly mechanical changes needed for a proper Dock widget and code >>>> shuffling: >>>> >>>> * Transform Spellchecker.ui into a simple QWidget >>>> * Pimpl all we can >>>> * Remove Close button (as for all other dock widgets) >>>> * Make the dialog buffer independent (as for all other dock widgets) >>>> >>>> Please give this patch a try and tell me if it's OK to commit. There are >>>> still quite some bugs for this dialog that I will try to address >>>> afterwards. >>> I tried your first patch and it is an improvement, thanks. >>> The dock widget obviously is the problem for the broken push button >>> placement too! >> >>> I'll test your second patch too, later today. >> >> Please test the latest (3rd?) patch instead. > > I did test it a little. It looks sane - but I've some strange effect: > sometimes the "ignore" button stops working, I have to switch between windows > to get it working again. Debugging it is tricky because it works when > it stops at some breakpoint and it doesn't happen all the time... > ... it happens if inside some inset (table cell, note or caption). > > I have to check if it's gone if I revert your patch...
Ok, the method to reproduce the problem is: * open Embedded manual * place the cursor inside the paragraph above the table in sub-section 2.6.2 Longtable alignment * start spell checker with F7 * hit ignore until the first cell with "asd" is reached * here the speller is stuck Without your patch it doesn't happen. The first reason why this is different may be the "missing" progress bar. Are you able to reproduce "my" problem? If not I can try to split your patch into two parts to find the culprit. Stephan
